Opsdir last call review of draft-ietf-curdle-cms-ecdh-new-curves-07

Stefan Winter <stefan.winter@restena.lu> Tue, 23 May 2017 09:29 UTC

MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Stefan Winter <stefan.winter@restena.lu>
To: ops-dir@ietf.org
Cc: draft-ietf-curdle-cms-ecdh-new-curves.all@ietf.org, curdle@ietf.org, ietf@ietf.org
Subject: Opsdir last call review of draft-ietf-curdle-cms-ecdh-new-curves-07
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <149553176028.29418.18414274218431090509@ietfa.amsl.com>
Date: Tue, 23 May 2017 02:29:20 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/ietf/uOKxhr5fG2uEQBMvuImRyZ87XGg>

Reviewer: Stefan Winter
Review result: Has Nits

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.

Opsdir last call review of draft-ietf-curdle-cms-… Stefan Winter
Re: Opsdir last call review of draft-ietf-curdle-… Russ Housley