Reviewer: Thomas Fossati Review result: Ready with Issues 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-emailcore-rfc5322bis-11 Reviewer: Thomas Fossati Review Date: 2024-04-21 IETF LC End Date: 2024-04-29 IESG Telechat date: Not scheduled for a telechat Summary: The document is clear and the changes from 5322 are fairly small-scale. There are some small changes to 5322's ABNF which look syntactically OK to me. However, running the ABNF checker on draft-ietf-emailcore-rfc5322bis-11.txt returns: * (234:23): error: This parser will truncate strings at %x00 * (236:28): error: This parser will truncate strings at %x00 * (238:25): error: This parser will truncate strings at %x00 * parsing failed: 3 errors encountered I don't understand what the error message is trying to tell me though. Re: IANA Considerations. There are changes in the registry schema and the registration template to add the trace indicator. The instructions are clear. Major issues: None Minor issues: 1. It'd be good to state the reasons why this document updates 3864 earlier than in §6.1. [1] recommends using the intro section for that. 2. Any reason for using the .test TLD rather than .example? RFC2606 says: ".test" is recommended for use in testing of current or new DNS related code. ".example" is recommended for use in documentation or as examples. [1] https://authors.ietf.org/required-content, Introduction checklist Nits/editorial comments: In §1.2.3: OLD: One reason for this latter requirement is that there are long-established sites on the Internet with mail archives that go back decades, archives with messages containing these elements. NEW: One reason for this latter requirement is that long-established Internet sites have mail archives dating back decades with messages containing these elements. In §3.6.4: OLD: Though listed as optional in the table (Table 1) in section 3.6 NEW: Though listed as optional in Table 1 of Section 3.6 -- last-call mailing list last-call@xxxxxxxx https://www.ietf.org/mailman/listinfo/last-call
