Last Call Review of draft-ietf-httpapi-deprecation-header-07
review-ietf-httpapi-deprecation-header-07-artart-lc-reschke-2024-09-10-00
Request | Review of | draft-ietf-httpapi-deprecation-header |
---|---|---|
Requested revision | No specific revision (document currently at 09) | |
Type | Last Call Review | |
Team | ART Area Review Team (artart) | |
Deadline | 2024-09-06 | |
Requested | 2024-08-23 | |
Authors | Sanjay Dalal , Erik Wilde | |
I-D last updated | 2024-09-10 | |
Completed reviews |
Genart Last Call review of -06
by Robert Sparks
(diff)
Artart Last Call review of -07 by Julian Reschke (diff) Secdir Telechat review of -08 by Robert Sparks (diff) |
|
Assignment | Reviewer | Julian Reschke |
State | Completed | |
Request | Last Call review on draft-ietf-httpapi-deprecation-header by ART Area Review Team Assigned | |
Posted at | https://mailarchive.ietf.org/arch/msg/art/hJICVEhAE2x5yJR-9W7a7gSmPZ4 | |
Reviewed revision | 07 (document currently at 09) | |
Result | Ready w/nits | |
Completed | 2024-09-10 |
review-ietf-httpapi-deprecation-header-07-artart-lc-reschke-2024-09-10-00
Nits: General: there's an *enormous* amount of repetition in the text. For instance, how many times does it say that the deprecation might be in the future or the past? Also, see end of Section 2.1. Section 2.1 Cites both the approved spec 8941bis and RFC 8941. "Deprecation = sf-date" comes surprising now that the ABNF reference is gone. Maybe just pull it into the prose, otherwise people not aware of ABNF might not know what this means. Example: maybe use UTC instead of GMT? Section 3 Introduction should link to RFC 8288. Section 4 "different data type" - the type actually is almost the same; maybe "format" would be clearer Section 6 Micro-nit: the templates use artwork format, they should be lists (will likely done by RFC Editor though) Section 8.1 The reference to RFC 9111 is only used in the implementation report, so by definition it can't be normative. I'd even remove it altogether from the reference section, given the fact the the implementation appendix will be removed. Section 8.2 For the same reason, I'd remove the reference to RFC7942.