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

Julian's feedback #350

Closed
mnot opened this issue May 23, 2017 · 2 comments
Closed

Julian's feedback #350

mnot opened this issue May 23, 2017 · 2 comments

Comments

@mnot
Copy link
Member

mnot commented May 23, 2017

2.1. About Intermediaries

An immutable response has the same semantic meaning when received by
proxy clients as it does when received by User-Agent based clients.
Therefore proxies SHOULD skip conditionally revalidating fresh
responses containing the immutable extension unless there is a signal
from the client that a validation is necessary (e.g. a no-cache
Cache-Control request directive).

Maybe point to Section 5.2.1.4 of RFC 7234 here...

A proxy that uses immutable to bypass a conditional revalidation may
choose whether to reply with a 304 or 200 to its requesting client
based on the request headers the proxy received.

s/may/MAY/ or s/may/can/

2.2. Example

Cache-Control: max-age=31536000, immutable

Maybe add a full example, including a request/response pair for the case when "immutable" is not present?

  1. IANA Considerations

[RFC7234] sections 7.1 and 7.1.2 require registration of the

s/[RFC7234] sections 7.1 and 7.1.2/Section 7.1 of .../

@mcmanus
Copy link
Contributor

mcmanus commented Jul 3, 2017

Maybe add a full example, including a request/response pair for the case when "immutable" is not present?

I actually toyed with this originally and decided the current arrangement was better. Drawing the absence of a request (or the case when nothing is different from the 723x state) didn't really clarify the normative text. It turned out better to just focus on the syntax of immutable in a response imo.

mcmanus added a commit that referenced this issue Jul 3, 2017
@mcmanus
Copy link
Contributor

mcmanus commented Jul 3, 2017

remainder addressed in -03. thanks

@mcmanus mcmanus closed this as completed Jul 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants