Skip to content

/ http-core

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

Jump to bottom

Misplaced text? #141

Closed
mnot opened this issue Oct 9, 2018 · 1 comment
Closed

Misplaced text? #141

mnot opened this issue Oct 9, 2018 · 1 comment
Labels
caching editorial

Comments

@mnot
Copy link
Member

@mnot mnot commented Oct 9, 2018

Note that cached responses that contain the "must-revalidate" and/or "s-maxage" response directives are not allowed to be served stale (Section 4.2.4) by shared caches. In particular, a response with either "max-age=0, must-revalidate" or "s-maxage=0" cannot be used to satisfy a subsequent request without revalidating it on the origin server.

This text seems out of place in 3.2 Storing Responses to Authenticated Requests. It also omits no-cache.
@mnot
Copy link
Member Author

@mnot mnot commented Oct 16, 2018

It seems like it's covered by 4.2.4 pretty completely.
mnot added a commit that referenced this issue Oct 16, 2018
@mnot
Remove unnecessary note
ca10689 
Fixes #141
@mnot mnot closed this in 29a469d Oct 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
caching editorial
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
@mnot