[Gen-art] Genart last call review of draft-ietf-kitten-pkinit-alg-agility-05

Christer Holmberg <christer.holmberg@ericsson.com> Fri, 01 March 2019 08:36 UTC

Return-Path: <christer.holmberg@ericsson.com>
X-Original-To: gen-art@ietf.org
Delivered-To: gen-art@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 1D97412D4F0; Fri, 1 Mar 2019 00:36:53 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Christer Holmberg <christer.holmberg@ericsson.com>
To: gen-art@ietf.org
Cc: kitten@ietf.org, ietf@ietf.org, draft-ietf-kitten-pkinit-alg-agility.all@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.92.1
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <155142941301.6207.9229636209459945659@ietfa.amsl.com>
Date: Fri, 01 Mar 2019 00:36:53 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/YX66FB4928EUINGlIkEWIwEvvZ4>
Subject: [Gen-art] Genart last call review of draft-ietf-kitten-pkinit-alg-agility-05
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.29
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: <https://mailarchive.ietf.org/arch/browse/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: Fri, 01 Mar 2019 08:36:53 -0000

Reviewer: Christer Holmberg
Review result: Ready with Issues

I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair.  Please treat these comments just
like any other last call comments.

For more information, please see the FAQ at

<https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.

Document: draft-ietf-kitten-pkinit-alg-agility-05
Reviewer: Christer Holmberg
Review Date: 2019-03-01
IETF LC End Date: 2019-02-17
IESG Telechat date: 2019-03-07

Summary: The draft needs to be more clear on how RFC 4556 is updated.

Major issues: None

Minor issues:

Q1:

For the unfamiliar reader, it is unclear exactly how RFC 4556 is updated. There
is e.g., no text indicating what sections of 4556 are updated, and there is no
"Updates to RFC 4556" section.

I assume the remainder of the draft then updates RFC 4556, but I think it needs
to be more clear for the unfamiliar reader.

Even if a "Updates to RFC 4556" section would not be feasible, the document
should indicate what parts/sections of 4556 are updated, and what sections in
the draft defines those updates.

Nits/editorial comments:

Q2:

Both the Abstract and the Introduction start with "This document updates
PKINIT,..."

I think it would be good to first give some background information on the
issue(s), and then state that the document fixes those issues by updating RFC
4556.