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 revision 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
I-D 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 revision 03 (document currently at 04)
Result Ready w/issues
Completed 2014-01-15
review-ietf-dane-registry-acronyms-03-genart-lc-romascanu-2014-01-15-00

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/