[kitten] Opsdir last call review of draft-ietf-kitten-pkinit-alg-agility-04

Scott Bradner <sob@sobco.com> Sun, 17 February 2019 20:21 UTC

Return-Path: <sob@sobco.com>
X-Original-To: kitten@ietf.org
Delivered-To: kitten@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 355FE128CB7; Sun, 17 Feb 2019 12:21:29 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Scott Bradner <sob@sobco.com>
To: ops-dir@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.91.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <155043488911.4083.7977373920397028733@ietfa.amsl.com>
Date: Sun, 17 Feb 2019 12:21:29 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/kitten/b9V7o2F3G-rs1gFARtamKx1zbxA>
Subject: [kitten] Opsdir last call review of draft-ietf-kitten-pkinit-alg-agility-04
X-BeenThere: kitten@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Common Authentication Technologies - Next Generation <kitten.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/kitten>, <mailto:kitten-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/kitten/>
List-Post: <mailto:kitten@ietf.org>
List-Help: <mailto:kitten-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/kitten>, <mailto:kitten-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sun, 17 Feb 2019 20:21:29 -0000

Reviewer: Scott Bradner
Review result: Has Nits

This is an OPS-DIR review of PKINIT Algorithm Agility
(draft-ietf-kitten-pkinit-alg-agility).  This ID updates PKINIT following the
guidelines in BCP 201 to make it algorithm agile and to expose acceptable
algorithms.

This is a very well written clear document.  I do not find any operational
issues with the document except that the document could use a section on
interoperability (old server-new client, new server-old client).

There are a few words about the topic deep in section 6 about the supportedKDFs
field but I do not see a general discussion or a discussion about the kdf
field. The text about the supportedKDFs field could be moved to a new section
and each of the old/new combinations could be addressed.  I see this as an
operational issue because compatibility issues are a frequent source of
operational headaches.