Re: [kitten] WGLC for three "bis" documents: draft-ietf-kitten-rfc4402bis-00, draft-ietf-kitten-rfc5653bis-01, draft-ietf-kitten-rfc6112bis-00

Nico Williams <nico@cryptonector.com> Mon, 16 February 2015 04:10 UTC

Return-Path: <nico@cryptonector.com>
X-Original-To: kitten@ietfa.amsl.com
Delivered-To: kitten@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 8546B1A8732 for <kitten@ietfa.amsl.com>; Sun, 15 Feb 2015 20:10:32 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 0.233
X-Spam-Level:
X-Spam-Status: No, score=0.233 tagged_above=-999 required=5 tests=[BAYES_20=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, IP_NOT_FRIENDLY=0.334, RCVD_IN_DNSWL_NONE=-0.0001] autolearn=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id tcwFnYB5iORC for <kitten@ietfa.amsl.com>; Sun, 15 Feb 2015 20:10:30 -0800 (PST)
Received: from homiemail-a107.g.dreamhost.com (sub4.mail.dreamhost.com [69.163.253.135]) by ietfa.amsl.com (Postfix) with ESMTP id 299171A19E4 for <kitten@ietf.org>; Sun, 15 Feb 2015 20:10:30 -0800 (PST)
Received: from homiemail-a107.g.dreamhost.com (localhost [127.0.0.1]) by homiemail-a107.g.dreamhost.com (Postfix) with ESMTP id D5E752005693E; Sun, 15 Feb 2015 20:10:29 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=cryptonector.com; h=date :from:to:cc:subject:message-id:references:mime-version :content-type:in-reply-to; s=cryptonector.com; bh=nt4kWzlZtGU60P NCsyPtQmC9+Q4=; b=BREtFZgKmVFTDcMq3FFzMMJBDKhB/aghkQJx/7EcESz628 FPULBkrBmFlGIZWKCW8B2WADuujJLQZqxoP8vT6nAzl4+rylkn99W9gvwq4/n+Zz xLKhIkrtQljv8gcsRZhyCBUdF0mfuPQACmqtvVkMi43+zs4KtGzuRfuU5B3Iw=
Received: from localhost (108-207-244-174.lightspeed.austtx.sbcglobal.net [108.207.244.174]) (Authenticated sender: nico@cryptonector.com) by homiemail-a107.g.dreamhost.com (Postfix) with ESMTPA id 84C2A20060013; Sun, 15 Feb 2015 20:10:29 -0800 (PST)
Date: Sun, 15 Feb 2015 22:10:28 -0600
From: Nico Williams <nico@cryptonector.com>
To: Benjamin Kaduk <kaduk@MIT.EDU>
Message-ID: <20150216041027.GB5246@localhost>
References: <alpine.GSO.1.10.1501201753140.23489@multics.mit.edu>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <alpine.GSO.1.10.1501201753140.23489@multics.mit.edu>
User-Agent: Mutt/1.5.21 (2010-09-15)
Archived-At: <http://mailarchive.ietf.org/arch/msg/kitten/v0zjD07PvH5nFUWGdcuUot3Q1k8>
Cc: kitten@ietf.org
Subject: Re: [kitten] WGLC for three "bis" documents: draft-ietf-kitten-rfc4402bis-00, draft-ietf-kitten-rfc5653bis-01, draft-ietf-kitten-rfc6112bis-00
X-BeenThere: kitten@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
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: <http://www.ietf.org/mail-archive/web/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: Mon, 16 Feb 2015 04:10:33 -0000

Please forgive me for being so late with these reviews.

On Tue, Jan 20, 2015 at 06:02:24PM -0500, Benjamin Kaduk wrote:
> http://tools.ietf.org/html/draft-ietf-kitten-rfc4402bis-00

FWIW, my only comment as to rfc4402bis is that rfc4402 wasn't "my" RFC,
just an Internet RFC that I authored.  Not sure if that's even a nit.

> http://tools.ietf.org/html/draft-ietf-kitten-rfc5653bis-01

I've not reviewed the entirety of this document, just the docs from
RFC5653 to rfc5653bis-02.

Comments:

My one comment on the changes is that having GSSException()
constructors that take major and minor status code numbers seems...
like a bad idea, particularly as to minor status codes.  HOWEVER,
since some implementations may use an all-Java GSSException class even
with a JNI bindings for the rest of the API (or for specific mechanism
providers), it seems like a good idea for that purpose.  I don't think
this is worth mentioning in the I-D; this is really just a comment for
the mailing list archive.

> http://tools.ietf.org/html/draft-ietf-kitten-rfc6112bis-00

Ditto here, I'm only reviewing the changes, not the entire I-D.
Although I have one comment as to the original RFC:

 - Should we take this opportunity to define an AD to carry the PKINIT
   client cert and its validation cert chain (and trust anchor)?

Regarding the change in section 4.2, the only reason for not reducing
the MUST further to MAY is probably interoperability with TGSes that
implemented the "MUST return a KRB-ERROR..." text that is being deleted.
This seems to merit a mention, something like:

   If the ticket in the PA-TGS-REQ of the TGS request is an anonymous
   one, the client SHOULD set the anonymous KDC option in the request
   for interoperability with TGSes that insist on it.  The TGS SHOULD
   ignore the presence/absence of the anonymous KDC option in the
   request when the the ticket in the request is an anonymous ticket.

(Do we need to be more careful about defining "anonymous ticket"?

 Clearly this refers to tickets whose cname is anonymous, but in the
 user-to-user case the sname of the second ticket could be anonymous,
 and when we add post-dating, one could even have a TGS-REQ where the
 ticket has an anonymous sname.  Such a request is bound to fail, and
 makes no sense anyways!  This is just a twisted case just to
 demonstrate that it's the cname of a ticket that determines whether its
 an anonymous one or not.  Just a curiosity.)

Nico
--