Skip to main content

Last Call Review of draft-ietf-emu-tls-eap-types-10
review-ietf-emu-tls-eap-types-10-genart-lc-fossati-2023-01-24-00

Request Review of draft-ietf-emu-tls-eap-types
Requested revision No specific revision (document currently at 13)
Type IETF Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2023-01-27
Requested 2023-01-13
Authors Alan DeKok
I-D last updated 2023-06-27 (Latest revision 2023-02-16)
Completed reviews Secdir IETF Last Call review of -11 by Melinda Shore (diff)
Genart IETF Last Call review of -10 by Thomas Fossati (diff)
Opsdir Telechat review of -11 by Jürgen Schönwälder (diff)
Intdir Telechat review of -11 by Bob Halley (diff)
Assignment Reviewer Thomas Fossati
State Completed
Request IETF Last Call review on draft-ietf-emu-tls-eap-types by General Area Review Team (Gen-ART) Assigned
Posted at https://mailarchive.ietf.org/arch/msg/gen-art/fNZOEID38s07l1DnsrSiqNpoXBc
Reviewed revision 10 (document currently at 13)
Result Ready
Completed 2023-01-24
review-ietf-emu-tls-eap-types-10-genart-lc-fossati-2023-01-24-00
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-emu-tls-eap-types-??
Reviewer: Thomas Fossati
Review Date: 2023-01-24
IETF LC End Date: 2023-01-27
IESG Telechat date: Not scheduled for a telechat

Summary:

The document adapts some existing EAP methods (EAP-FAST, EAP-TTLS, TEAP
and PEAP) to be used with TLS 1.3.  In particular: 
- it defines the new key derivations,
- clarifies the use of client certs,
- describes the rules around processing of application data,
  NewSessionTicket and resumption

The draft is very clearly written and organised.

The IANA section contains clear, unambiguous instructions.

From a gen-art perspective it looks ready to me.

Major issues: none

Minor issues: none

Nits/editorial comments:

OLD
   There remain some differences between EAP-TLS and other TLS-based EAP
   methods which necessitates this document.
NEW
   There remain some differences between EAP-TLS and other TLS-based EAP
   methods which are addressed by this document.

OLD
   Similarly, when the inner authentication protocol indicates that
   authentication has succeed
NEW
   Similarly, when the inner authentication protocol indicates that
   authentication has succeeded

Style (feel free to ignore):
- "all of" => all
- "both of [these]" => "both"
- "whether or not" => "whether"

Typos:
- indicatoin
- deriviation
- identies