Last Call Review of draft-ietf-dnsop-dns-tcp-requirements-12
review-ietf-dnsop-dns-tcp-requirements-12-artart-lc-mahoney-2021-09-03-00
Request | Review of | draft-ietf-dnsop-dns-tcp-requirements |
---|---|---|
Requested revision | No specific revision (document currently at 15) | |
Type | Last Call Review | |
Team | ART Area Review Team (artart) | |
Deadline | 2021-09-03 | |
Requested | 2021-08-20 | |
Authors | John Kristoff , Duane Wessels | |
I-D last updated | 2021-09-03 | |
Completed reviews |
Tsvart Last Call review of -12
by Mirja Kühlewind
(diff)
Artart Last Call review of -12 by Jean Mahoney (diff) Secdir Last Call review of -12 by Alan DeKok (diff) Genart Last Call review of -12 by Dan Romascanu (diff) Intdir Telechat review of -13 by Ron Bonica (diff) |
|
Assignment | Reviewer | Jean Mahoney |
State | Completed | |
Request | Last Call review on draft-ietf-dnsop-dns-tcp-requirements by ART Area Review Team Assigned | |
Posted at | https://mailarchive.ietf.org/arch/msg/art/YAoRhQIrKn4g0L6XFZvV45bWL44 | |
Reviewed revision | 12 (document currently at 15) | |
Result | Ready w/nits | |
Completed | 2021-09-03 |
review-ietf-dnsop-dns-tcp-requirements-12-artart-lc-mahoney-2021-09-03-00
Reviewer: Jean Mahoney Review result: Ready with nits A well-written, easy-to-read document. Love Appendix A! Question about Appendix A.2 and Updates - Should this document also update RFC 1536? Current text in A.2: The informational document [RFC1536] states UDP is the "chosen protocol for communication though TCP is used for zone transfers." That statement should now be considered in its historical context and is no longer a proper reflection of modern expectations. Nits: General - Document status (Informational, Standards Track, etc.) should be capitalized, and Standards Track is not hyphenated (There's just one instance of hyphenation). Section 2.4 - 35%of / 35% of Section 3 - transport.[TDNS] / transport [TDNS]. Section 5.1 Current: "the steady-state of lost resources as a result is a 'DNS wedgie'." Perhaps: "the steady state of the resulting lost resources is a 'DNS wedgie'." Section 5.2 - Expand the acronym KSK. Section 7 - The Acknowledgments section should be located just above the Authors' Addresses section. It looks like the names are supposed to be in alphabetical order, but they aren't quite. Section 9 - fragmenetation / fragmentation Section 10 - Since DNS over UDP and TCP use / Since DNS over UDP and TCP uses Section 11.2 - [ROLL_YOU_ROOT] has a mangled author name and a TBD. Appendix A - The construction "The [RFCNNNN] document..." (in A.3, A.4, A.5, A.7, and A.13) reads oddly to me. Perhaps "This document [RFCNNNN] ". Appendix A.8 - The verb tenses are mixed in this section. Appendix A.32 - as a a / as a There are other nits I could pick more easily if this doc was in a GitHub repo. They can be left to the RPC to clean up. :-) Thanks! Jean