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 www.strayalpha.com > On Mar 1, 2022, at 7:44 AM, Lars Eggert <l...@eggert.org> 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 <nore...@ietf.org> 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 >> t...@ietf.org >> https://www.ietf.org/mailman/listinfo/tcpm >
_______________________________________________ Gen-art mailing list Gen-art@ietf.org https://www.ietf.org/mailman/listinfo/gen-art