Last Call Review of draft-ietf-curdle-cms-ecdh-new-curves-07
review-ietf-curdle-cms-ecdh-new-curves-07-opsdir-lc-winter-2017-05-23-00
| Request | Review of | draft-ietf-curdle-cms-ecdh-new-curves |
|---|---|---|
| Requested revision | No specific revision (document currently at 10) | |
| Type | Last Call Review | |
| Team | Ops Directorate (opsdir) | |
| Deadline | 2017-05-28 | |
| Requested | 2017-05-14 | |
| Authors | Russ Housley | |
| I-D last updated | 2017-05-23 | |
| Completed reviews |
Secdir Last Call review of -07
by Charlie Kaufman
(diff)
Genart Last Call review of -07 by Roni Even (diff) Opsdir Last Call review of -07 by Stefan Winter (diff) Genart Telechat review of -09 by Roni Even (diff) |
|
| Assignment | Reviewer | Stefan Winter |
| State | Completed | |
| Request | Last Call review on draft-ietf-curdle-cms-ecdh-new-curves by Ops Directorate Assigned | |
| Reviewed revision | 07 (document currently at 10) | |
| Result | Has nits | |
| Completed | 2017-05-23 |
review-ietf-curdle-cms-ecdh-new-curves-07-opsdir-lc-winter-2017-05-23-00
Nits: * 2.1 starts with "... based on a one-way hash function described in ANS X9.63 [X963]." s/ANS/ANSI/ * Chapter 7 defines six OIDs (secg-scheme 11 1...3 and smime-alg TBD1...3) and also includes the base OIDs under which these new OIDs are attached (secg-scheme and smime-alg). Those two are not defined in this document, but the text in chapter 7 suggests so. Maybe it would be a bit clearer if the text stated explicitly which of the OIDs in the chapter are NEW, and which ones already exist and are provided for reference/context.