Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add cache-status and cdn-cache-control header name constants #583

Merged
merged 2 commits into from
Feb 17, 2023

Conversation

acfoltzer
Copy link
Member

Docs include pointers to the relevant RFCs, as these are not in the base HTTP semantics spec.

Docs include pointers to the relevant RFCs, as these are not in the base HTTP semantics spec.
@seanmonstar
Copy link
Member

I think this issue would be relevant here: #573

@acfoltzer
Copy link
Member Author

@seanmonstar thanks for the tip. I made some recommendations over in that thread. Since both of these are part of the IANA registry as non-provisional entries, I think we should definitely include them, but perhaps we should should add all the non-provisional registry names in one go?

@seanmonstar
Copy link
Member

I was trying to read up on these headers, I didn't remember what they were. That's because they're new (2022)! Their respective RFCs claim their current status is "Proposed Standard". Are they used elsewhere? I don't seem to notice them on sites behind Cloudflare, Cloudfront, or GCP.

@acfoltzer
Copy link
Member Author

Yeah, they're still relatively new, but are a standardization of a scattering of implementation-defined headers like Surrogate-Control and X-Cache. FWIW, once something makes it to the Proposed Standard state, things like header names are very unlikely to change (thus the permanent status for these in the IANA field name registry), it's mostly about nailing down details of edge cases at that point.

@seanmonstar seanmonstar merged commit 1fad3ea into hyperium:master Feb 17, 2023
@acfoltzer acfoltzer deleted the cache-header-names branch February 21, 2023 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants