Last Call Review of draft-ietf-tls-esni-23
review-ietf-tls-esni-23-genart-lc-bryant-2025-03-18-00

Request Review of draft-ietf-tls-esni
Requested revision No specific revision (document currently at 24)
Type IETF Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2025-03-13
Requested 2025-02-20
Authors Eric Rescorla , Kazuho Oku , Nick Sullivan , Christopher A. Wood
I-D last updated 2025-03-23 (Latest revision 2025-03-20)
Completed reviews Dnsdir IETF Last Call review of -23 by R. (Miek) Gieben (diff)
Artart IETF Last Call review of -23 by Carsten Bormann (diff)
Secdir IETF Last Call review of -23 by Adam W. Montville (diff)
Tsvart IETF Last Call review of -23 by Tommy Pauly (diff)
Genart IETF Last Call review of -23 by Stewart Bryant (diff)
Opsdir IETF Last Call review of -24 by Giuseppe Fioccola
Dnsdir Telechat review of -24 by R. (Miek) Gieben
Intdir Telechat review of -24 by Tommy Pauly
Assignment Reviewer Stewart Bryant
State Completed
Request IETF Last Call review on draft-ietf-tls-esni by General Area Review Team (Gen-ART) Assigned
Posted at https://mailarchive.ietf.org/arch/msg/gen-art/RADHmnh2Ir3GJoqcjTGtjgtWf8Y
Reviewed revision 23 (document currently at 24)
Result Ready w/nits
Completed 2025-03-18
review-ietf-tls-esni-23-genart-lc-bryant-2025-03-18-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://wiki.ietf.org/en/group/gen/GenArtFAQ>.

Document: draft-ietf-tls-esni-23
Reviewer: Stewart Bryant
Review Date: 2025-03-18
IETF LC End Date: 2025-03-13
IESG Telechat date: Not scheduled for a telechat

Summary:A well written document with some minor nits that are easily addressed.

Major issues: None

Minor issues: None

Nits/editorial comments:

   fields, such as the ALPN list [RFC7301].  Co-located servers with
SB> ALPN needs expanding on first use.
========

   or they send a GREASE ECH 
SB> I believe that GREASE is an acronym and should be expanded.
========

(see Section 2 of
   [DNS-TERMS]).  
SB> ID-NITS identifies the following concern:
  -- Obsolete informational reference (is this intentional?): RFC 8499 (ref.
     'DNS-TERMS') (Obsoleted by RFC 9499)
Should the reference be changed?
=========

   Note that, if the cookie includes a key name, analogous to Section 4
   of [RFC5077], this may leak information if different backend servers
   issue cookies with different key names at the time of the connection.

SB> From ID-NITS
  -- Obsolete informational reference (is this intentional?): RFC 5077
     (Obsoleted by RFC 8446)

Should the reference be changed?