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 | |
| Draft 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 | |
| Review |
review-ietf-curdle-cms-chacha20-poly1305-04-opsdir-lc-comstedt-2016-12-24
|
|
| 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