Last Call Review of draft-ietf-dane-registry-acronyms-03
review-ietf-dane-registry-acronyms-03-genart-lc-romascanu-2014-01-15-00

Request Review of draft-ietf-dane-registry-acronyms
Requested rev. no specific revision (document currently at 04)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2014-01-23
Requested 2014-01-09
Authors Ólafur Guðmundsson
Draft last updated 2014-01-15
Completed reviews Genart Last Call review of -03 by Dan Romascanu (diff)
Genart Telechat review of -03 by Dan Romascanu (diff)
Secdir Last Call review of -03 by Tobias Gondrom (diff)
Assignment Reviewer Dan Romascanu
State Completed
Review review-ietf-dane-registry-acronyms-03-genart-lc-romascanu-2014-01-15
Reviewed rev. 03 (document currently at 04)
Review result Ready with Issues
Review completed: 2014-01-15

Review
review-ietf-dane-registry-acronyms-03-genart-lc-romascanu-2014-01-15






I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, please see the FAQ at




 




<

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




 




Please resolve these comments along with any other Last Call comments you may receive.




 




Document: http://www.ietf.org/id/draft-ietf-dane-registry-acronyms-03.txt




Reviewer: Dan Romascanu




Review Date: 1/15/2014




IETF LC End Date: 1/23/2014




IESG Telechat date:




 




Summary:




Ready with issues




 




Major issues:




None – it’s a simple, clear and useful document. 




 




Minor issues:







1.

      


In Section 2.3 the I-D recommends that the values in reference column for SHA-256 and SHA-512 refer to [RFC 6698] while the IANA Considerations section in RFC 6698 recommends and the registry entries in the TLSA
 Matching Types table at 


http://www.iana.org/assignments/dane-parameters/dane-parameters.xhtml

  point to [RFC 6234].







2.

      


As this I-D updates the registries with a column for acronyms, it seems more accurate that the reference columns of all tables mention both  [RFC 6698] and [RFC XXXX] (this RFC)




 




Nits/editorial comments:




In the Introduction section: 




‘This document updates the IANA registry definition for TLSA




   record to add a column with acronym for each specified field’





s/acronym/an acronym/