Skip to content
This repository was archived by the owner on Nov 22, 2018. It is now read-only.

Port fix for Update how certain cache directives are handled to 1.1.x #91

Closed
JunTaoLuo opened this issue Jan 10, 2017 · 5 comments
Closed

Comments

@JunTaoLuo
Copy link
Contributor

We'll need to port back part of the fixes in #88 back to 1.1.x.

At the minimum we'll need the fixes to:

Also potentially consider

  • handling of proxy-revalidate cache directive and handling of max-stale without specified limit
@JunTaoLuo JunTaoLuo added this to the 1.1.1 milestone Jan 10, 2017
@JunTaoLuo
Copy link
Contributor Author

cc @muratg

@Eilon
Copy link
Member

Eilon commented Jan 19, 2017

This patch bug is approved. Please use the normal code review process w/ a PR and make sure the fix is in the correct branch, then close the bug and mark it as done.

@Eilon
Copy link
Member

Eilon commented Jan 19, 2017

@muratg please assign.

@JunTaoLuo
Copy link
Contributor Author

Patch merged. Leaving the issue open since version numbers will need to be updated.

@JunTaoLuo
Copy link
Contributor Author

Changes merged. Updating version numbers now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants