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

Rebecca VanRheenen <rvanrheenen@amsl.com> Mon, 15 April 2024 05:58 UTC

Return-Path: <rvanrheenen@amsl.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 89D22C14E515; Sun, 14 Apr 2024 22:58:52 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.897
X-Spam-Level:
X-Spam-Status: No, score=-1.897 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001, URIBL_DBL_BLOCKED_OPENDNS=0.001, URIBL_ZEN_BLOCKED_OPENDNS=0.001] autolearn=unavailable autolearn_force=no
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 iCZWlC09MKyX; Sun, 14 Apr 2024 22:58:47 -0700 (PDT)
Received: from c8a.amsl.com (c8a.amsl.com [4.31.198.40]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 85779C14F61D; Sun, 14 Apr 2024 22:58:47 -0700 (PDT)
Received: from localhost (localhost [127.0.0.1]) by c8a.amsl.com (Postfix) with ESMTP id 2C442424B427; Sun, 14 Apr 2024 22:58:47 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
Received: from c8a.amsl.com ([127.0.0.1]) by localhost (c8a.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id nWSxtRgJiObD; Sun, 14 Apr 2024 22:58:47 -0700 (PDT)
Received: from [IPv6:2601:641:300:5fb0:71a8:ebef:6a0e:dd19] (unknown [IPv6:2601:641:300:5fb0:71a8:ebef:6a0e:dd19]) by c8a.amsl.com (Postfix) with ESMTPSA id CA560424B426; Sun, 14 Apr 2024 22:58:46 -0700 (PDT)
Content-Type: text/plain; charset="utf-8"
Mime-Version: 1.0 (Mac OS X Mail 13.4 \(3608.120.23.2.7\))
From: Rebecca VanRheenen <rvanrheenen@amsl.com>
In-Reply-To: <5ba407e6-c38c-4c5a-aa90-2ee968277ee4@igalia.com>
Date: Sun, 14 Apr 2024 22:58:45 -0700
Cc: Alanna Paloma <apaloma@amsl.com>, RFC Editor <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>
Content-Transfer-Encoding: quoted-printable
Message-Id: <AEB58485-FBCC-432E-BFC2-FE7879259CC7@amsl.com>
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> <5ba407e6-c38c-4c5a-aa90-2ee968277ee4@igalia.com>
To: Ujjwal Sharma <ryzokuken@igalia.com>, Carsten Bormann <cabo@tzi.org>
X-Mailer: Apple Mail (2.3608.120.23.2.7)
Archived-At: <https://mailarchive.ietf.org/arch/msg/auth48archive/-J_Zy6Pm8eEcrtYM-e8tm4LoEbc>
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: Mon, 15 Apr 2024 05:58:52 -0000

Hi Ujjwal,

Thank you for your comments. We made the updates you suggest (see list of files below) but have some followup questions as well:


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

Would deleting “For this" be an improvement? Or do you have another suggestion?

Current:
   [RFC3339] defines a syntax for timestamps to represent date and time
   in the Internet.  For this, the present document defines an extension
   syntax that achieves the following properties:

Perhaps:
   [RFC3339] defines a syntax for timestamps to represent date and time
   in the Internet.  The present document defines an extension
   syntax that achieves the following properties:


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

Please discuss with Carsten and let us know how to update this. We see that the changes to this defintion were proposed by Carsten in response to question #4 in the initial set of questions.

Original:
   Z:  A suffix which, when applied to a time, denotes a UTC offset of
      00:00; often spoken "Zulu" from the ICAO phonetic alphabet
      representation of the letter "Z".  (Definition from Section 2 of
      [RFC3339]; see [ICAO-PA] for the phonetic alphabet.)

Current:
   Z:  A suffix that, when applied to a time, denotes a UTC offset of
      00:00; often pronounced "Zulu" from the ICAO phonetic alphabet
      representation of the letter "Z".  (The definition is from
      Section 2 of [RFC3339]; see the International Civil Aviation
      Organization (ICAO) document [ICAO-PA] for the phonetic alphabet
      mentioned.)


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)

For the AUTH48 status of this document, please see:
  https://www.rfc-editor.org/auth48/rfc9557

Thank you,
RFC Editor/rv



> On Apr 12, 2024, at 6:26 AM, Ujjwal Sharma <ryzokuken@igalia.com> wrote:
> 
> 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
>