Re: [auth48] AUTH48: RFC-to-be 9557 <draft-ietf-sedate-datetime-extended-11> for your review

Ujjwal Sharma <ryzokuken@igalia.com> Fri, 12 April 2024 13:26 UTC

Return-Path: <ryzokuken@igalia.com>
X-Original-To: auth48archive@ietfa.amsl.com
Delivered-To: auth48archive@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 744C1C14F5F9; Fri, 12 Apr 2024 06:26:24 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.097
X-Spam-Level:
X-Spam-Status: No, score=-2.097 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_DBL_BLOCKED_OPENDNS=0.001, URIBL_ZEN_BLOCKED_OPENDNS=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=igalia.com
Received: from mail.ietf.org ([50.223.129.194]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id l4ZrBnWNjv57; Fri, 12 Apr 2024 06:26:20 -0700 (PDT)
Received: from fanzine2.igalia.com (fanzine.igalia.com [178.60.130.6]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id AC2FEC14F5F5; Fri, 12 Apr 2024 06:26:19 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=igalia.com; s=20170329; h=Content-Transfer-Encoding:Content-Type:In-Reply-To:From: References:Cc:To:Subject:MIME-Version:Date:Message-ID:Sender:Reply-To: Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender: Resent-To:Resent-Cc:Resent-Message-ID:List-Id:List-Help:List-Unsubscribe: List-Subscribe:List-Post:List-Owner:List-Archive; bh=ftJaulZ+ohUj3deAY2LEr+icaY7RkeOfEldFmol0tTM=; b=mIU8V0j4EQwzth3EtTImfv2pW/ pFM0gwDDRVRBTyWh3Ci2os0AlyIkVXHzbZfiY/SlBeU1d2TWyBAbd+HT5Frxg63QpfoCQdbbVXYUn 1zW+cptnwNG9Jyr2jz+B4U9ndwzB560a62vaFODu3GpPg/t8JMzRnsdMEIdb/PrqTHAeVVReeURfq xU4U27Q+/UNgYqxECclX/1YjLcSzhU77AFhbQlkRxCyhRdlP0dMixJS7XLLz40fNNdZQwMwMP9n4f axHJn4VzvxKNYSosGpdZDcVO1wuOv1Sfaowfq3Rxri9dxeEZ1h+3JD9TQDEsag18lVog9fZpee/jh +5tPROFA==;
Received: from static-14-111-7-89.ipcom.comunitel.net ([89.7.111.14] helo=[192.168.0.18]) by fanzine2.igalia.com with esmtpsa (Cipher TLS1.3:ECDHE_X25519__RSA_PSS_RSAE_SHA256__AES_128_GCM:128) (Exim) id 1rvGux-003uCl-SN; Fri, 12 Apr 2024 15:26:11 +0200
Message-ID: <5ba407e6-c38c-4c5a-aa90-2ee968277ee4@igalia.com>
Date: Fri, 12 Apr 2024 15:26:11 +0200
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
To: Alanna Paloma <apaloma@amsl.com>
Cc: Carsten Bormann <cabo@tzi.org>, RFC Errata System <rfc-editor@rfc-editor.org>, sedate-ads@ietf.org, sedate-chairs@ietf.org, Mark McFadden <Mark@internetpolicyadvisors.com>, Francesca Palombini <francesca.palombini@ericsson.com>, "auth48archive@rfc-ed" <auth48archive@rfc-editor.org>
References: <20240312020857.BB09C191A4A3@rfcpa.amsl.com> <5F5DE402-52EB-4D24-AF8A-D6467917DF11@tzi.org> <CD607473-2982-4D5B-AF6E-E4BED86DB5AC@amsl.com> <A725C2FF-A365-4D86-8F10-C86F1E72676E@tzi.org> <93B26FD7-B3EA-4EFA-AB2C-A34AE416C7CB@amsl.com> <F6596E8C-B5BE-43E7-A6E4-6C7D157A51BB@tzi.org> <EA3026D6-E0A1-4897-86D1-62B3BFE0BD7E@amsl.com> <5FEB859A-86E7-4E18-BA10-2D09E6CC3539@amsl.com> <16BB7AD0-8BA8-4E9D-B149-5AF04CCBDFF1@tzi.org> <874CEDAD-68A2-4904-9B6E-F2FFB2C72FE5@amsl.com> <6681B520-8184-45DC-9636-B992093EC5FB@tzi.org> <2490729F-E726-425F-9120-02810C26F6DB@amsl.com> <7D4E4AB4-8E73-47A5-B7FD-F271187ABB16@amsl.com> <e57c84c6-51d8-415f-87dd-ec5bf45d45a1@igalia.com> <2AF15CEB-0156-42B3-8535-27FBBDA19FE4@amsl.com>
Content-Language: en-US
From: Ujjwal Sharma <ryzokuken@igalia.com>
Organization: Igalia S.L.
In-Reply-To: <2AF15CEB-0156-42B3-8535-27FBBDA19FE4@amsl.com>
Content-Type: text/plain; charset="UTF-8"; format="flowed"
Content-Transfer-Encoding: 8bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/auth48archive/Ser2y0D27C00WOKd_z4Fkg4TvsU>
Subject: Re: [auth48] AUTH48: RFC-to-be 9557 <draft-ietf-sedate-datetime-extended-11> for your review
X-BeenThere: auth48archive@rfc-editor.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: "Archiving AUTH48 exchanges between the RFC Production Center, the authors, and other related parties" <auth48archive.rfc-editor.org>
List-Unsubscribe: <https://mailman.rfc-editor.org/mailman/options/auth48archive>, <mailto:auth48archive-request@rfc-editor.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/auth48archive/>
List-Post: <mailto:auth48archive@rfc-editor.org>
List-Help: <mailto:auth48archive-request@rfc-editor.org?subject=help>
List-Subscribe: <https://mailman.rfc-editor.org/mailman/listinfo/auth48archive>, <mailto:auth48archive-request@rfc-editor.org?subject=subscribe>
X-List-Received-Date: Fri, 12 Apr 2024 13:26:24 -0000

Thanks Alanna!

While I proceed with the review, I have a batch of comments already.

Comments:

* In Scope, the "For this, …" starting the second sentence seems 
awkwardly worded.

* In Scope, the "it does not address" list should probably use "or" 
rather than "and".

* In Definitions, for UTC, I disagree with introduction of the comma in 
"…in conjunction with leap seconds , as announced by…" (but I don't feel 
strongly about this).

* In Definitions, for UTC Offset, I think past tense would be more 
appropriate than present tense (i.e., "…in the wintertime in 2023 was 5 
hours behind UTC, so its UTC offset was -05:00.").

* In Definitions, for Z, I disagree with altering the exact literal text 
of RFC 3339 while claiming that document as its source.

* In Inconsistent time-offset and Time Zone Information, the second use 
of "in Figure 1" should probably be stricken for redundancy and the 
containing sentence could be reworded for clarity to something like 
"…where the first case is one for which the application MUST act on the 
inconsistency (the time zone suffix is marked critical) and the second 
case is one for which it MAY act on the inconsistency (the time zone 
suffix is elective).".

* In ABNF, "which both are explicitly excluded" should be reworded to 
"which are both explicitly excluded".

* In ABNF, for the final note, I disagree with the change from "an entry 
in a timezone database on another platform" to "an entry in a Time Zone 
Database on another platform"—the new capitalization suggests reference 
to the [proper, singular] IANA Time Zone Database, which is not the 
case. I would recommend instead rewording that to "a time zone name on 
another platform" along with rewording "As the time-zone-name will 
ultimately have to be looked up in the database" to "As the 
time-zone-name will ultimately have to be looked up in the local database".

Hope they sound good, feel free to ask for any clarification.

Cheers,
Ujjwal

On 12/04/2024 12:06 a.m., Alanna Paloma wrote:
> Hi Ujjwal,
> 
> Thank you for letting us know. We’ve made a note of this on the AUTH48 status page:
> https://www.rfc-editor.org/auth48/rfc9557
> 
> We look forward to hearing back from you when your review is complete.
> 
> Best regards,
> RFC Editor/ap
> 
>> On Apr 10, 2024, at 9:45 AM, Ujjwal Sharma <ryzokuken@igalia.com> wrote:
>>
>> My apologies for the delay Alanna, I've known that the publication is currently blocked on my final approval and that gives me no joy. However, I've been stuck with some rather complicated issues at work and therefore couldn't get to it despite trying really hard. I'll let you know as soon as I'm done.
>>
>> Best,
>> Ujjwal
>>
>> On 09/04/2024 6:39 p.m., Alanna Paloma wrote:
>>> Hi Ujjwal,
>>> This is a friendly reminder that we await your review and approval of this document.
>>> The files have been posted here (please refresh):
>>> https://www.rfc-editor.org/authors/rfc9557.txt
>>> https://www.rfc-editor.org/authors/rfc9557.pdf
>>> https://www.rfc-editor.org/authors/rfc9557.html
>>> https://www.rfc-editor.org/authors/rfc9557.xml
>>> The relevant diff files are posted here:
>>> https://www.rfc-editor.org/authors/rfc9557-diff.html (comprehensive diff)
>>> https://www.rfc-editor.org/authors/rfc9557-auth48diff.html (all AUTH48 changes)
>>> https://www.rfc-editor.org/authors/rfc9557-lastdiff.html (htmlwdiff diff between last version and this)
>>> https://www.rfc-editor.org/authors/rfc9557-lastrfcdiff.html (rfcdiff between last version and this)
>>> Once we’ve received your approval, we with move forward with the publication process.
>>> For the AUTH48 status of this document, please see:
>>> https://www.rfc-editor.org/auth48/rfc9557
>>> Thank you,
>>> RFC Editor/ap
>>>> On Apr 2, 2024, at 3:24 PM, Alanna Paloma <apaloma@amsl.com> wrote:
>>>>
>>>> Hi Carsten,
>>>>
>>>> We have updated the files per your request and noted your approval on the AUTH48 status page:
>>>> https://www.rfc-editor.org/auth48/rfc9557
>>>>
>>>> The files have been posted here (please refresh):
>>>> https://www.rfc-editor.org/authors/rfc9557.txt
>>>> https://www.rfc-editor.org/authors/rfc9557.pdf
>>>> https://www.rfc-editor.org/authors/rfc9557.html
>>>> https://www.rfc-editor.org/authors/rfc9557.xml
>>>>
>>>> The relevant diff files are posted here:
>>>> https://www.rfc-editor.org/authors/rfc9557-diff.html (comprehensive diff)
>>>> https://www.rfc-editor.org/authors/rfc9557-auth48diff.html (all AUTH48 changes)
>>>> https://www.rfc-editor.org/authors/rfc9557-lastdiff.html (htmlwdiff diff between last version and this)
>>>> https://www.rfc-editor.org/authors/rfc9557-lastrfcdiff.html (rfcdiff between last version and this)
>>>>
>>>> Once we’ve received Ujjwal’s approval, we with move forward with the publication process.
>>>>
>>>> Thank you,
>>>> RFC Editor/ap
>>>>
>>>>> On Apr 2, 2024, at 2:02 PM, Carsten Bormann <cabo@tzi.org> wrote:
>>>>>
>>>>> Hi Alanna,
>>>>>
>>>>> thank you.
>>>>>
>>>>> I have one final change that is needed:
>>>>>
>>>>> OLD:
>>>>>   reference point for the specified time"; see <xref target="update"/>8.</t>
>>>>> NEW:
>>>>>   reference point for the specified time"; see <xref target="update"/>.</t>
>>>>>
>>>>> Apologies for not noticing this earlier.
>>>>>
>>>>>
>>>>> Also, the addition of an article here was incomplete (generator and recipient are not the same):
>>>>>
>>>>> OLD:
>>>>> As a generator and recipient may be using different revisions of the
>>>>> NEW:
>>>>> As a generator and a recipient may be using different revisions of the
>>>>>
>>>>>
>>>>> With these changes, I believe RFC 9557-to-be will be ready for publication.
>>>>>
>>>>> Grüße, Carsten
>>>>>
>>>>>
>>>>>> On 2. Apr 2024, at 18:03, Alanna Paloma <apaloma@amsl.com> wrote:
>>>>>>
>>>>>> Hi Carsten,
>>>>>>
>>>>>> Thank you for the explanation. We will leave the original phrasing of the text as is.
>>>>>>
>>>>>> As all of our queries have now been addressed, we will await any further updates you may have as well as approvals from each party listed on the AUTH48 status page below prior to moving this document forward in the publication process.
>>>>>>
>>>>>> For the AUTH48 status of this document, please see:
>>>>>> https://www.rfc-editor.org/auth48/rfc9557
>>>>>>
>>>>>> Thank you,
>>>>>> RFC Editor/ap
>>>>> […]
>>>>
>>>>
>>
>> -- 
>> Ujjwal "Ryzokuken" Sharma (he/him)
>>
>> Compilers Hacker at Igalia
>>
> 
> 

-- 
Ujjwal "Ryzokuken" Sharma (he/him)

Compilers Hacker at Igalia