Skip to content
New issue

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

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

Already on GitHub? # to your account

Allow defining a cache as "shared" #167

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ionrock
Copy link
Contributor

@ionrock ionrock commented Aug 7, 2017

In the spec (https://tools.ietf.org/html/rfc7234#section-5.2.2.6) it talks
about the private directive and that a shared cache MUST NOT store
the response.

This starts by defining the cache as being "shared" and testing for
the "shared" attribute on the cache.

Fixes: #141

In the spec (https://tools.ietf.org/html/rfc7234#section-5.2.2.6) it talks
about the `private` directive and that a shared cache MUST NOT store
the response.

This starts by defining the cache as being "shared" and testing for
the "shared" attribute on the cache.

Fixes: #141
@ionrock ionrock force-pushed the ionrock/define-shared-caches branch from 2df4ce1 to a53442d Compare August 7, 2017 14:08
@Flameeyes
Copy link
Contributor

FTR, I have something along these lines in #225 as well.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Proper Cache-Control: private support
2 participants