Last Call Review of draft-ietf-curdle-cms-ecdh-new-curves-07
review-ietf-curdle-cms-ecdh-new-curves-07-genart-lc-even-2017-05-25-00

Request Review of draft-ietf-curdle-cms-ecdh-new-curves
Requested rev. no specific revision (document currently at 09)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2017-05-28
Requested 2017-05-14
Other Reviews Secdir Last Call review of -07 by Charlie Kaufman (diff)
Opsdir Last Call review of -07 by Stefan Winter (diff)
Genart Telechat review of -09 by Roni Even
Review State Completed
Reviewer Roni Even
Review review-ietf-curdle-cms-ecdh-new-curves-07-genart-lc-even-2017-05-25
Posted at https://mailarchive.ietf.org/arch/msg/gen-art/xiqxrAXihrb3UTzMufOW6Ab9NZE
Reviewed rev. 07 (document currently at 09)
Review result Ready with Nits
Last updated 2017-05-25

Review
review-ietf-curdle-cms-ecdh-new-curves-07-genart-lc-even-2017-05-25

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

<https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.

Document: draft-ietf-curdle-cms-ecdh-new-curves-??
Reviewer: Roni Even
Review Date: 2017-05-25
IETF LC End Date: 2017-05-28
IESG Telechat date: Not scheduled for a telechat

Summary:
The document is ready for publication as a standard track RFC

Major issues:

Minor issues:

Nits/editorial comments: 

In general it was easy to read and follow.  Maybe it will be good to repeat in section 3.2 the definition of KeyAgreeRecipientInfo . This is done is section 2 for ECC-CMS-SharedInfo but it is not crucial.

1. There is no ToC
2. In section 2.2 fourth paragraph "is used two places" - "in two .."