Thank you, Ben!
— Carlos Pignataro, https://carlos.pignataro.net <https://carlos.pignataro.net/> “Sometimes I use big words that I do not fully understand, to make myself sound more photosynthesis." > On Apr 19, 2022, at 1:59 PM, Ben Schwartz > <bemasc=40google....@dmarc.ietf.org> wrote: > > > > On Tue, Apr 19, 2022 at 11:30 AM Carlos Pignataro via Datatracker > <nore...@ietf.org <mailto:nore...@ietf.org>> wrote: > Reviewer: Carlos Pignataro > Review result: Ready with Nits > > I am an assigned INT directorate reviewer for draft-ietf-dnsop-svcb-https. > These comments were written primarily for the benefit of the Internet Area > Directors. Document editors and shepherd(s) should treat these comments just > like they would treat comments from any other IETF contributors and resolve > them along with any other Last Call comments that have been received. For more > details on the INT Directorate, see > https://datatracker.ietf.org/group/intdir/about/ > <https://datatracker.ietf.org/group/intdir/about/> > <https://datatracker.ietf.org/group/intdir/about/ > <https://datatracker.ietf.org/group/intdir/about/>>. > > This document specifies the "SVCB" and "HTTPS" DNS resource record (RR) types > to facilitate the lookup of information needed to make connections to network > services, such as for HTTP origins. > > This document is clear and comprehensive. Specifically, rev -08 improved > structure, terminology, and readability. It seems ready modulo some nits. > > Editorial suggestions: > * s/followup/follow-up/g > * " * Fallback to the the client's non-Alt-Svc connection behavior" --> > duplicate "the" * "IANA from the "Resource Record (RR) TYPEs" subregistry" --> > looks like that's a registry, not sub-registry (same with other parts of the > IANA sections) * s/e.g./e.g.,/g * "2 octet field" -> "2-octet field" > > Thanks for flagging these style issues. I've proposed these changes in a > Github Pull Request [1], except for the comma changes (which appear to be a > contested point of English style). > > [1] https://github.com/MikeBishop/dns-alt-svc/pull/390 > <https://github.com/MikeBishop/dns-alt-svc/pull/390> > > Editorially, I find this construct a bit interesting -- indirections, and > wonder if a bit more context can be added to S7.5. 7.5. "mandatory" > See Section 8. > 8. ServiceMode RR compatibility and mandatory keys > > Yes, this is a bit odd, as is the forward reference for "ech". However, it > avoids disrupting the flow of Section 7 with extensive subsections, and helps > to highlight important material. > > I have no strong opinions about how this ought to be laid out in the document. > > Thank you for your consideration, > > Carlos. > > > -- > last-call mailing list > last-c...@ietf.org <mailto:last-c...@ietf.org> > https://www.ietf.org/mailman/listinfo/last-call > <https://www.ietf.org/mailman/listinfo/last-call>
signature.asc
Description: Message signed with OpenPGP
_______________________________________________ DNSOP mailing list DNSOP@ietf.org https://www.ietf.org/mailman/listinfo/dnsop
