Med,
0. "This document obsoletes RFC 7125." -> It would have been good to ask the authors of that document to review the new draft. 1. Introduction "The bits in offsets 0 through 3 are not header flags, but the TCP segment Data Offset field." -> This paragraph appears out of context, and it doesn't need to be indented. "and, therefore, [RFC7125] risks to deviate from the authoritative TCP registry [TCP-FLAGS]." -> s/to deviate/deviating/ 3. The tcpControlBits Information Element It's not immediately clear whether this section is showing the current definition or proposing a new definition. A title of "Updated / Proposed / Revised tcpControlBits Information Element" would be quite clear. "by the collector. Use the tcpHeaderLength Information Element to encode this value." -> s/collector/Collector/ I appreciate this language is copied from the existing definition, but "Collector" is a defined IPFIX term and this is the only IE in the registry which uses lowercase. 5. Security Considerations "This document uses a stronger requirement language compared to [RFC7125]." -> s/requirement/requirements/ P. _______________________________________________ OPSAWG mailing list [email protected] https://www.ietf.org/mailman/listinfo/opsawg
