Telechat Review of draft-ietf-dnsop-svcb-https-08
review-ietf-dnsop-svcb-https-08-intdir-telechat-pignataro-2022-04-19-00
| Request | Review of | draft-ietf-dnsop-svcb-https |
|---|---|---|
| Requested revision | No specific revision (document currently at 09) | |
| Type | Telechat Review | |
| Team | Internet Area Directorate (intdir) | |
| Deadline | 2022-02-27 | |
| Requested | 2022-01-31 | |
| Requested by | Éric Vyncke | |
| Authors | Benjamin M. Schwartz , Mike Bishop , Erik Nygren | |
| Draft last updated | 2022-04-19 | |
| Completed reviews |
Genart Last Call review of -07
by
Dale R. Worley
(diff)
Artart Last Call review of -07 by Cullen Jennings (diff) Opsdir Last Call review of -07 by Joe Clarke (diff) Tsvart Last Call review of -07 by Kyle Rose (diff) Intdir Telechat review of -08 by Carlos Pignataro (diff) |
|
| Comments |
This I-D is important for the DNS community as it will have an impact also on ADD WG. A fresh view from outside the DNS community would be a plus probably. Thanks. |
|
| Assignment | Reviewer | Carlos Pignataro |
| State | Completed | |
| Review |
review-ietf-dnsop-svcb-https-08-intdir-telechat-pignataro-2022-04-19
|
|
| Posted at | https://mailarchive.ietf.org/arch/msg/int-dir/99T6XTBi-gmByLfag4RgHyOfOMw | |
| Reviewed revision | 08 (document currently at 09) | |
| Result | Ready with Nits | |
| Completed | 2022-04-19 |
review-ietf-dnsop-svcb-https-08-intdir-telechat-pignataro-2022-04-19-00
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/>. 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" 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 Thank you for your consideration, Carlos.