Re: [Emu] draft-ietf-emu-eap-tls13-16.txt

Alan DeKok <aland@deployingradius.com> Mon, 14 June 2021 00:21 UTC

Return-Path: <aland@deployingradius.com>
X-Original-To: emu@ietfa.amsl.com
Delivered-To: emu@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 8E4853A168B for <emu@ietfa.amsl.com>; Sun, 13 Jun 2021 17:21:46 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.899
X-Spam-Level:
X-Spam-Status: No, score=-1.899 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=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 3JZqk2ltpnwP for <emu@ietfa.amsl.com>; Sun, 13 Jun 2021 17:21:42 -0700 (PDT)
Received: from mail.networkradius.com (mail.networkradius.com [62.210.147.122]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 43A723A168A for <emu@ietf.org>; Sun, 13 Jun 2021 17:21:41 -0700 (PDT)
Received: from [192.168.46.129] (24-52-251-6.cable.teksavvy.com [24.52.251.6]) by mail.networkradius.com (Postfix) with ESMTPSA id EA513389; Mon, 14 Jun 2021 00:21:38 +0000 (UTC)
Authentication-Results: NetworkRADIUS; dmarc=none (p=none dis=none) header.from=deployingradius.com
From: Alan DeKok <aland@deployingradius.com>
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: quoted-printable
Mime-Version: 1.0 (Mac OS X Mail 13.4 \(3608.120.23.2.6\))
Date: Sun, 13 Jun 2021 20:21:37 -0400
References: <CAOgPGoBXRAABeC_kCcCrsUPC03e8C_GGpzJHB+aWAue5sE=9zw@mail.gmail.com> <4789411B-9D6A-4A33-B465-DCEC2369E671@deployingradius.com> <BA5BC7E9-EC6F-4A10-9F19-284572AF2710@deployingradius.com> <ac3fda5a-65ef-0e57-fdb0-fffdc08bb9e1@ericsson.com> <4F473CF1-CE5C-4834-AF7E-7FCB2457B199@deployingradius.com> <CAOgPGoCoFRF-0uowFiVRtDyWATzr0d3pNKz7DJo5CDBesiDP5Q@mail.gmail.com>
To: Joseph Salowey <joe@salowey.net>, EMU WG <emu@ietf.org>
In-Reply-To: <CAOgPGoCoFRF-0uowFiVRtDyWATzr0d3pNKz7DJo5CDBesiDP5Q@mail.gmail.com>
Message-Id: <7B957BC5-DEC4-48D6-A032-C8AC1CBFD210@deployingradius.com>
X-Mailer: Apple Mail (2.3608.120.23.2.6)
Archived-At: <https://mailarchive.ietf.org/arch/msg/emu/64K-weTxKtG4bKI3NQ98rbdiYok>
Subject: Re: [Emu] draft-ietf-emu-eap-tls13-16.txt
X-BeenThere: emu@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "EAP Methods Update \(EMU\)" <emu.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/emu>, <mailto:emu-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/emu/>
List-Post: <mailto:emu@ietf.org>
List-Help: <mailto:emu-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/emu>, <mailto:emu-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 14 Jun 2021 00:21:47 -0000

On Jun 13, 2021, at 7:49 PM, Joseph Salowey <joe@salowey.net> wrote:
> [Joe]  The existing text already refers to relying on the underlying TLS version negotiation.  

  Yes.  I was trying to address the confusion between version negotiation and other handshake negotiation.

> [Joe]  I don't see a problem with covering new TLS handshake messages in the document, however they are covered somewhat inconsistently.  Perhaps we should cover them all in a "new handshake messages section". 

  My point was that if the EAP layer doesn't need to do anything with those new handshake messages, then there isn't a need to mention them in the specification.

> TLS 1.3 introduces several new handshake messages including HelloRetryRequest, NewSessionTicket, KeyUpdate.  In general, these messages will be handled by the underlying TLS libraries and are not visible to EAP-TLS, however there are a few things to note.  
> 
> The HelloRetryRequest is used by the server to reject the parameters offered in the ClientHello and suggest new parameters.  When this message is encountered it will increase the number of round trips used by the protocol.  

  If that's the only change, then I would suggest there's no need for a section dedicated to HelloRetryRequest.  Perhaps just saying "there are a number of new messages in TLS 1.3 which affect round trips, but the EAP layer doesn't have to do anything other than exchange more TLS messages".

> [Joe]  OK how about adding:
> 
> "An example of limiting network access would be to invoke a vendor's walled garden or quarantine network functionality."

  Sounds good.

> >>   I don't understand why it's necessary to include discussion of TLS negotiation in EAP, when that negotiation does not affect EAP in any way.
> > See above.
> 
>   In short: TLS version negotiation does affect EAP-TLS.  HelloRetryRequest messages do not.
> 
> [Joe] The EAP TLS layer needs to know which TLS version was negotiated.  HelloRetryRequests affect the number of round trips in the exchange, but are opaque with respect to EAP-TLS.

  Yes.  I was confused at Section 2.1.6. which essentially says "HelloRetryRequest is a new TLS message, here's a diagram of using it with EAP-TLS".   As an implementor, the immediate question is "What do I *do* with this message?"  and the answer is essentially "nothing".

  Pretty much every other section in the document says "this is new in TLS 1.3, and here's how to handle it".

> [Joe]  HelloRetryRequest is a feature of the underlying TLS library so the RFC8446 Should determine the underlying behavior, specifying a different behavior is problematic. 

  Sure.

> >>   The updated text is clearer, but still does not address some of the questions raised below about calling this out as a new requirement from 5216, and what happens in an HA environment.
> > Please let us know a good reference to "implementation and 
> > interoperability experience." about the upper limit on EAP packet 
> > exchanges. This would also be useful for draft-ietf-emu-eaptlscert.
> 
>   I posted references on this list previously:
> 
> https://mailarchive.ietf.org/arch/browse/emu/?qdr=y
> 
> [Joe] I think this is the wrong link.  

  Yes.  The new mail archive interface is substantially worse than the older "mailman" interface.  It's difficult to follow threads, quickly search by dates, etc.

  In any case, this is the previous message:  https://mailarchive.ietf.org/arch/msg/emu/y3d6KGFeImAeSLEqF6XqVMev2Zc/

> >>> Requiring a supplicant to be configured with a peer name is a new requirement from RFC 5216, and isn't called out as such.
> > I agree with you that we could benefit from more clarification on 
> > relationship between section 2.2 and 5.2 of this draft vs. the old RFC 
> > 5216. Do you have a suggestion how best to split the text and reflect 
> > the updates to RFC5216?
> 
>   Updating 2.2 with a note after the text on page 18 seems best:
> 
> While this requirement to verify domain names is new, and was not mentioned in [RFC5216], it has been widely implemented in EAP peers.  As such, we believe that this section contains minimal new interoperability or implementation requirements on EAP peers.
> 
> [Joe] This does not seem to add to the specification. 

  I agree that text doesn't add any new requirements to the specification.  However, it's useful to make a note for implementors that while Section 2.2. is officially a new requirement in theory, there is little to do in practice, because implementations already meet these requirements.  There has been similar text in many other specifications.

  Alan DeKok.