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 | IETF 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 | 2025-03-17 (Latest revision 2024-09-27) | |
| Completed reviews |
Genart IETF Last Call review of -06
by Robert Sparks
(diff)
Artart IETF 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 | IETF 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.