Skip to main content

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