Skip to main content

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.