Skip to main content

Last Call Review of draft-ietf-pce-pceps-tls13-02
review-ietf-pce-pceps-tls13-02-genart-lc-holmberg-2023-12-08-00

Request Review of draft-ietf-pce-pceps-tls13
Requested revision No specific revision (document currently at 04)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2023-12-19
Requested 2023-12-05
Authors Dhruv Dhody , Sean Turner , Russ Housley
I-D last updated 2023-12-08
Completed reviews Genart Last Call review of -02 by Christer Holmberg (diff)
Rtgdir Early review of -02 by Tal Mizrahi (diff)
Assignment Reviewer Christer Holmberg
State Completed
Request Last Call review on draft-ietf-pce-pceps-tls13 by General Area Review Team (Gen-ART) Assigned
Posted at https://mailarchive.ietf.org/arch/msg/gen-art/mDGxPjiRujzmd8fI8fipLwOl0Po
Reviewed revision 02 (document currently at 04)
Result Almost ready
Completed 2023-12-08
review-ietf-pce-pceps-tls13-02-genart-lc-holmberg-2023-12-08-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-pce-pceps-tls13-02
Reviewer: Christer Holmberg
Review Date: 2023-12-08
IETF LC End Date: 2023-12-19
IESG Telechat date: Not scheduled for a telechat

Summary: The document is well written, and easy to understand. I do have one
Minor issue/question and a few Editorial issues/questions that I would like the
authors to address.

Major issues: N/A

Minor issues:

Q1:Section 3 adds text saying that PCEPS implementations MUST NOT use early
data, and there are a couple of notes about what early data is. However, I
cannot find text which explains the "MUST NOT use". If the case where early
media is permitted does not apply to PCEPS it would be good to add text which
explains it. It would also be good to explain the reason in the Introduction of
this document.

Nits/editorial comments:

Q2:In a few places the text says "TLS protocol", and in other places "TLS".
Would it be possible to use "TLS" everywhere?

Q3: Section 6 indicates that there are no known implementations when version
-02 of the draft was posted. If that is still the case when the RFC is
published, could the whole section be removed?

Q4: Related to Q3, if the section remains (e.g., because there are known
implementations), I suggest to say "time of publishing this document" instead
of "time of posting of this Internet-Draft".