-
Notifications
You must be signed in to change notification settings - Fork 13
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
Purge cache_key if response headers do not include content digest #22
Conversation
Looks good to me! @olleolleolle any chance you can help me with a detailed review? |
@olleolleolle anything else pending on this? |
@evman182 Thanks for asking! I had to reconstruct the "why do we do this?", and my reconstitution of it is: "We purge bodies for which the response was not created with an Is that a nearly-correct way of expressing the reasoning for the change? |
@olleolleolle it's already purigng the meta entries if the bodies don't exist for the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your helpful PR!
Would it be possible to cut a release? |
Yes, I have been working on that. |
No description provided.