Skip to main content

Last Call Review of draft-ietf-curdle-cms-chacha20-poly1305-04
review-ietf-curdle-cms-chacha20-poly1305-04-opsdir-lc-comstedt-2016-12-24-00

Request Review of draft-ietf-curdle-cms-chacha20-poly1305
Requested revision No specific revision (document currently at 06)
Type Last Call Review
Team Ops Directorate (opsdir)
Deadline 2016-12-16
Requested 2016-12-02
Authors Russ Housley
I-D last updated 2016-12-24
Completed reviews Opsdir Last Call review of -04 by Niclas Comstedt (diff)
Genart Last Call review of -04 by Matthew A. Miller (diff)
Secdir Last Call review of -04 by Yoav Nir (diff)
Opsdir Telechat review of -05 by Niclas Comstedt (diff)
Genart Telechat review of -05 by Matthew A. Miller (diff)
Assignment Reviewer Niclas Comstedt
State Completed
Request Last Call review on draft-ietf-curdle-cms-chacha20-poly1305 by Ops Directorate Assigned
Reviewed revision 04 (document currently at 06)
Result Has nits
Completed 2016-12-24
review-ietf-curdle-cms-chacha20-poly1305-04-opsdir-lc-comstedt-2016-12-24-00
Hi,

I have reviewed this document as part of the Operational directorate's ongoing
effort to review all IETF documents being processed by the IESG.  These
comments were written with the intent of improving the operational aspects of
the IETF drafts. Comments that are not addressed in last call may be included
in AD reviews during the IESG review.  Document editors and WG chairs should
treat these comments just like any other last call comments.

Document reviewed: draft-ietf-curdle-cms-chacha20-poly1305-04

Caveat: This is outside my usual area and I have limited direct operational
experience.

Summary: Minor nits

- Section 1, first paragraph. I wonder if this sentence can be re-written to
make it a little easier and less repetitive on [CMS] references - Section 3,
change the 2nd must to all capitals in the following sentence "The
AlgorithmIdentifier parameters field MUST be present, and the parameters field
must contain a AEADChaCha20Poly1305Nonce:”

/nco