Skip to main content

Last Call Review of draft-ietf-dnsop-maintain-ds-03
review-ietf-dnsop-maintain-ds-03-genart-lc-sparks-2016-07-08-00

Request Review of draft-ietf-dnsop-maintain-ds
Requested revision No specific revision (document currently at 06)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2016-07-11
Requested 2016-06-30
Authors Ólafur Guðmundsson , Paul Wouters
I-D last updated 2016-07-08
Completed reviews Genart Last Call review of -03 by Robert Sparks (diff)
Secdir Last Call review of -03 by Benjamin Kaduk (diff)
Assignment Reviewer Robert Sparks
State Completed
Request Last Call review on draft-ietf-dnsop-maintain-ds by General Area Review Team (Gen-ART) Assigned
Reviewed revision 03 (document currently at 06)
Result Ready w/nits
Completed 2016-07-08
review-ietf-dnsop-maintain-ds-03-genart-lc-sparks-2016-07-08-00
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.

For more information, please see the FAQ at

<

http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

Document: draft-ietf-dnsop-maintain-ds-03
Reviewer: Robert Sparks
Review Date: 8 Jul 2016
IETF LC End Date: 11 Jul 2016
IESG Telechat date: Not yet scheduled for a telechat

Summary: Ready, but with nits and perhaps a process problem

Potential process problem:

This document intends to move RFC7344 from Informational to PS in place
(without republishing RFC7344. The intent to do so is buried at the end
of the document (the abstract doesn't mention it). The Last Call for the
document does not make it clear that _this_ document is elevating RFC7344.
(It at least mentions it, which is good, but the writeup about the elevation
can be read to say "we're considering this elevation somewhere else, keep it
in mind while evaluating this document").

There is no hint from the subject line that this is a call to bring RFC7344
onto the standards track. Unless there is some other communication effort
that I've missed on a quick search, I think it is very likely that most
of the IETF community outside the dnsop working group missed this intent.
I strongly encourge a last call focusing _specifically_ on moving RFC7344
to the standards track without republication.

My personal feedback on elevating RFC7344 without republishing is that it's
not the right thing to do. At the very least "Category: Informational"
appears in the document itself, and that will not change. If the IESG
decides to proceed with this as currently formulated, count me in the
deep rough.

Nits:

In 1.2, "that decision SHOULD be fully under the child domain's control"...
Why is that a 2119 SHOULD? I think this is commentary on that it would be
a bad idea for someone else to unilaterally decide to turn of DNSSEC for
a child domain? Why not just say that (it would be even better to expand
on _why_ it's a bad idea. If you really think this is the right way to say
what you mean, and you keep 2119, please talk about when it would be ok to
not follow that SHOULD.

In 1.3, consider pointing to Appendix A of RFC7344 to better define RRR.

In the Security Considerations, you have "Users SHOULD" and "all options
SHOULD be considered". These are not meaningul uses of 2119 - please use
prose to say what you really mean. If you want to keep them, please talk
about when it would be ok to not follow the SHOULD. I think you're trying


to say "Completing the rollover via an unsigned state is dangerous and 


should



only be used as a last resort" or something similarly strong.



Consider pointing back to the 5 scenarios you spell out in section 1.2 


in the



security considerations section. The asserted existance of operational and


aoftware limitations that necessitate turning off DNSSEC to facilitate a 


change



of operator is certainly a major security consideration.

Consider doing more to the DNS Security Algorithms Number registry than


the current instructions indicate. Simply adding a reference to this 


document


to the row for number 0 does not convey that this "reserved" number is 


actually


being _used_ in a protocol, and that when it is it's an algorithm number 


that



is not a number for an algorithm. I don't know how to say that cleanly, but


the registry should say more than simply "reserved" if this document is 


approved.




Typo-nit: s/digiest/digest/