[Gen-art] Gen-ART LC Review of draft-jivsov-openpgp-ecc-10

Christer Holmberg <christer.holmberg@ericsson.com> Mon, 19 March 2012 12:07 UTC

Return-Path: <christer.holmberg@ericsson.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost []) by ietfa.amsl.com (Postfix) with ESMTP id 4CCC121F8688 for <gen-art@ietfa.amsl.com>; Mon, 19 Mar 2012 05:07:20 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -10.259
X-Spam-Status: No, score=-10.259 tagged_above=-999 required=5 tests=[AWL=0.339, BAYES_00=-2.599, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-8]
Received: from mail.ietf.org ([]) by localhost (ietfa.amsl.com []) (amavisd-new, port 10024) with ESMTP id Sz5bHSbT7ekX for <gen-art@ietfa.amsl.com>; Mon, 19 Mar 2012 05:07:18 -0700 (PDT)
Received: from mailgw10.se.ericsson.net (mailgw10.se.ericsson.net []) by ietfa.amsl.com (Postfix) with ESMTP id 974C421F867B for <gen-art@ietf.org>; Mon, 19 Mar 2012 05:07:17 -0700 (PDT)
X-AuditID: c1b4fb3d-b7c6fae0000045c0-e5-4f67217499ee
Received: from esessmw0256.eemea.ericsson.se (Unknown_Domain []) by mailgw10.se.ericsson.net (Symantec Mail Security) with SMTP id 1E.48.17856.471276F4; Mon, 19 Mar 2012 13:07:16 +0100 (CET)
Received: from ESESSCMS0356.eemea.ericsson.se ([]) by esessmw0256.eemea.ericsson.se ([]) with mapi; Mon, 19 Mar 2012 13:07:16 +0100
From: Christer Holmberg <christer.holmberg@ericsson.com>
To: "gen-art@ietf.org" <gen-art@ietf.org>, "draft-jivsov-openpgp-ecc.all@tools.ietf.org" <draft-jivsov-openpgp-ecc.all@tools.ietf.org>
Date: Mon, 19 Mar 2012 13:07:15 +0100
Thread-Topic: Gen-ART LC Review of draft-jivsov-openpgp-ecc-10
Thread-Index: Ac0Fw76iBDA3ZcbzRWOLmDvbxqOt+w==
Message-ID: <7F2072F1E0DE894DA4B517B93C6A05852C413D2304@ESESSCMS0356.eemea.ericsson.se>
Accept-Language: en-US
Content-Language: en-US
acceptlanguage: en-US
Content-Type: multipart/alternative; boundary="_000_7F2072F1E0DE894DA4B517B93C6A05852C413D2304ESESSCMS0356e_"
MIME-Version: 1.0
X-Brightmail-Tracker: AAAAAA==
Subject: [Gen-art] Gen-ART LC Review of draft-jivsov-openpgp-ecc-10
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 19 Mar 2012 12:07:20 -0000

I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, please see the FAQ at <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

Please resolve these comments along with any other Last Call comments you may receive.

Document:                                         draft-jivsov-openpgp-ecc-10

Reviewer:                                           Christer Holmberg

Review Date:                                     2012-03-19

IETF LC End Date:                             2012-04-09

IESG Telechat date: (if known)   2012-04-09

Summary:           The draft is ready for publication, with a couple of editorial nits.

Major issues:     -

Minor issues:     -

Nits/editorial comments:

Q1:         The "MPI" abbreviation is used throughout the document, but it is not expanded anywhere, nor is there any reference.

Q2:         The Abstract says:

                "The document aims to standardize an optimal but narrow set of parameters for best interoperability"

I think it would to make it more clear what interoperability you refer to.

Q3:         The Abstract says:

                "within the framework it defines"

                However, the framework is not mentioned/clarified/defined anywhere in the document, so if you want to talk about a framework I think some clarification is needed.

Q4:         In the Introduction, please add references to OpenPGP, RSA and DSA. Also consider expansion on first occurrence.

Q5:         The document uses "This document", "This specification", "This draft" and "This standard" terminology. Please use consistent terminology.

Q6:         In section 5, I don't think you need the "defined in" parts. Simply add the reference.

                Example:             "Elliptic Curve Digital Signature Algorithm (ECDSA) [FIPS 186-3],..."

                Maybe similar changes can be done in some other parts of the document also.

Q7:         In section 5, consider modifying the following sentence:

                "The section 9.1. Public-Key Algorithms of [RFC4880] is expanded to define the following public key algorithm IDs"


                "This section extends section 9.1 (Public-Key Algorithms) or [RFC4880], by defining the following public key algorithm IDs"

Q8:         Section 5 says:

                "Applications MUST support ECDSA and ECDH."

                Please clarify what applications you refer to. I assume you mean "Applications that implement this specification", or something similar...

                The same comment also applies to section 12.1, and section 13 (the word is used many times in section 13, but it is enough to clarify it on first occurrence :)