Last Call Review of draft-ietf-tls-iana-registry-updates-04

Request Review of draft-ietf-tls-iana-registry-updates
Requested rev. no specific revision (document currently at 05)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2018-03-01
Requested 2018-02-15
Authors Joseph Salowey, Sean Turner
Draft last updated 2018-02-20
Completed reviews Secdir Last Call review of -04 by Stefan Santesson (diff)
Opsdir Last Call review of -04 by Dan Romascanu (diff)
Genart Last Call review of -04 by Stewart Bryant (diff)
Assignment Reviewer Stewart Bryant 
State Completed
Review review-ietf-tls-iana-registry-updates-04-genart-lc-bryant-2018-02-20
Reviewed rev. 04 (document currently at 05)
Review result Ready with Issues
Review completed: 2018-02-20


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


Document: draft-ietf-tls-iana-registry-updates-05
Reviewer: Stewart Bryant
Review Date: 2018-02-20
IETF LC End Date: 2018-03-01
IESG Telechat date: 2018-03-08

Summary: A well written document that is difficult to check and easy to make a mistake with. There are a tiny number of editorial matters. The matter of the semantics of Recommended = no may need to further thought and clarification.

Major issues: None

Minor issues:

I think convention is to list the documents being updated in the Abstract, but cannot find any formal guidance.


  If an item is marked as not recommended it does not necessarily mean
SB> Do you mean "marked as not recommended" or "not marked as recommended".

SB>  I am worried about the semantics of Recommended = no.
SB> Presumably there are three states: recommended, not recommended, 
SB> and silent/don't know/don't care/not yet. Which of these
SB> states does Recommended = no represent?

Nits/editorial comments: 

   This document describes a number of changes to (D)TLS IANA registries

SB> TLS is not a well known abbreviation and so needs expanding


   This document instructs IANA to make changes to a number of (D)TLS-

SB> TLS needs expanding