[Last-Call] Re: [DNSOP] Genart last call review of draft-ietf-dnsop-generalized-notify-05

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Peter,

On 2/12/25 12:08, Peter Yee via Datatracker wrote:
Reviewer: Peter Yee
Review result: Ready with Nits

I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair.  Please treat these comments just
like any other last call comments.

Thank you for your review!

Minor issues:

Page 12, section 6.2, 1st paragraph, 1st sentence: I could not find the “DSYNC:
Location of Synchronization of Endpoints” registry. Contrast that with section
6.1, where the requested type (DSYNC) is already registered! While the wording
in section 6.1 doesn’t hurt, the wording in 6.2 could indicate to IANA that no
further action is needed upon publication of this document as an RFC, which
would not appear to be the case.

Ah, indeed these sections are worded inconsistently. We've fixed it by making 6.2 follow the style of 6.1 / 6.3.

IANA has already confirmed which actions they think need to be done, and it's all correct (see 2025-02-05 at https://datatracker.ietf.org/doc/draft-ietf-dnsop-generalized-notify/history/).

Nits/editorial comments:

Page 4, section 2.1, Scheme, 2nd sentence: change “8 bit” to “8-bit”.

Page 4, section 2.1, Port, 2nd sentence: change “16 bit to 16-bit”.

Page 10, section 4.3, item 1, 2nd paragraph, 1st sentence: change “occuring” to
“occurring”.

Page 10, section 4.3, item 1, 3rd paragraph, 2nd sentence: append a comma after
“e.g.”.

Page 13, Purpose: change “Use case” to “Use-case”.

We've followed all these suggestions (but the last).

Thanks again,
Peter + co-authors

--
last-call mailing list -- last-call@xxxxxxxx
To unsubscribe send an email to last-call-leave@xxxxxxxx




[Index of Archives]     [IETF Annoucements]     [IETF]     [IP Storage]     [Yosemite News]     [Linux SCTP]     [Linux Newbies]     [Mhonarc]     [Fedora Users]

  Powered by Linux