Last Call Review of draft-ietf-ippm-checksum-trailer-03
review-ietf-ippm-checksum-trailer-03-genart-lc-romascanu-2015-10-22-00
Request | Review of | draft-ietf-ippm-checksum-trailer |
---|---|---|
Requested revision | No specific revision (document currently at 06) | |
Type | Last Call Review | |
Team | General Area Review Team (Gen-ART) (genart) | |
Deadline | 2015-10-28 | |
Requested | 2015-10-14 | |
Authors | Tal Mizrahi | |
I-D last updated | 2015-10-22 | |
Completed reviews |
Genart Last Call review of -03
by Dan Romascanu
(diff)
Genart Telechat review of -05 by Dan Romascanu (diff) Genart Telechat review of -05 by Dan Romascanu (diff) Secdir Last Call review of -03 by Rich Salz (diff) Opsdir Last Call review of -03 by Sarah Banks (diff) |
|
Assignment | Reviewer | Dan Romascanu |
State | Completed | |
Request | Last Call review on draft-ietf-ippm-checksum-trailer by General Area Review Team (Gen-ART) Assigned | |
Reviewed revision | 03 (document currently at 06) | |
Result | Ready w/issues | |
Completed | 2015-10-22 |
review-ietf-ippm-checksum-trailer-03-genart-lc-romascanu-2015-10-22-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 < http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>. Document: draft-ietf-ippm-checksum-trailer-03 Reviewer: Dan Romascanu Review Date: 10/22/15 IETF LC End Date: N/A IESG Telechat date: N/A Summary: This document is ready for publication. I have a couple of minor issues that need clarification and can be solved easily by minor editing. Major issues: Minor issues: 1. It is not clear to me how backwards compatibility is ensured. How do implementations make distinction between OWAMP or TWAMP packets that use timestamp update and Checksum Complement rather than timestamp update and UDP checksum update? Is a mix of senders/receivers (for OWAMP) or senders/reflectors (for TWAMP) that some support and some do not support Checksum Complement possible? 2. The last paragraph in section 4 reads: ‘The concept described in this document is intended to be used only in unauthenticated or in authenticated mode.’ This seems either a mistake, or I did not understand what it means and some clarification is needed. Nits/editorial comments: