Re: [Last-Call] [tcpm] Genart last call review of draft-ietf-tcpm-ao-test-vectors-05

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi, Peter,

Thank you for the review.

Most of these issues were addressed already in the -06 version.

Regarding “master key”:
RFC 5925 uses the term as a two-word phrase, capitalized only at the beginning of sentences
RFC 5926 uses the term Master_Key (note the underscore) as an algorithmic parameter

This doc uses the 5926 version and will be consistent to that. I have added a note in -07 (to be posted shortly) accordingly.

Joe

Dr. Joe Touch, temporal epistemologist

On Mar 1, 2022, at 7:44 AM, Lars Eggert <lars@xxxxxxxxxx> wrote:

Peter, thank you for your review. I have entered a No Objection ballot for this document.

Lars


On 2022-1-30, at 23:25, Peter Yee via Datatracker <noreply@xxxxxxxx> wrote:

Reviewer: Peter Yee
Review result: Ready with Nits

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-tcpm-ao-test-vectors-05
Reviewer: Peter Yee
Review Date: 2022-01-30
IETF LC End Date: 2022-02-01
IESG Telechat date: Not scheduled for a telechat

Summary: This document supplies test vectors for the MTI authentication
algorithms used in the TCP Authentication Option. There are a few nits that
should be corrected prior to publication. I'm not equipped to validate the test
vectors themselves and did not attempt to do so. [Ready with nits.]

Major issues: None

Minor issues: None

Nits/editorial comments:

Page 2, Abstract: move this entire section before the "Status of this Memo"
section, as required by RFC 7322, section 4.

Page 4, first full paragraph: move "[RFC6978]" after "experimental extension"
to avoid confusion.

Page 10, section 5, 1st paragraph: add a period at the end of the sentence.

Page 20, section 7, 1st sentence: add a period at the end of the sentence.

Page 24, section 8.2: add a comma after "e.g.".

Page 25, section 8.4, 2nd bullet item: change "to" to "in".

Page 25, section 8.4, 3rd bullet item: change first "to" to "in".

Page 25, section 9, 2nd paragraph: between the usage in section 3.1.1 and here,
choose a consistent case for "master_key". I know, RFC 5926 (Master_Key) and
RFC 5925 (master_key) are not consistent between themselves.



_______________________________________________
tcpm mailing list
tcpm@xxxxxxxx
https://www.ietf.org/mailman/listinfo/tcpm


-- 
last-call mailing list
last-call@xxxxxxxx
https://www.ietf.org/mailman/listinfo/last-call

[Index of Archives]     [IETF Annoucements]     [IETF]     [IP Storage]     [Yosemite News]     [Linux SCTP]     [Linux Newbies]     [Mhonarc]     [Fedora Users]

  Powered by Linux