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

Turn off default caching of POST requests #190

Open
jpd236 opened this issue May 18, 2018 · 0 comments
Open

Turn off default caching of POST requests #190

jpd236 opened this issue May 18, 2018 · 0 comments
Milestone

Comments

@jpd236
Copy link
Collaborator

jpd236 commented May 18, 2018

Original context: #154

Volley caches POST requests by default. But the semantics for doing so are pretty broken - the request body is not included in the default cache key and POST requests are always supposed to hit the origin server according to the RFC - the cached response should only be used for subsequent GET requests in certain scenarios (e.g. https://www.mnot.net/blog/2012/09/24/caching_POST).

Unless/until we have an actual implementation of a working POST cache by default, we should disable this functionality and leave it up to clients to implement or turn on if it's really what they want.

@jpd236 jpd236 added this to the 1.2.0 milestone May 18, 2018
@jpd236 jpd236 modified the milestones: 1.2.0, 1.2.1 Feb 15, 2021
@jpd236 jpd236 modified the milestones: 1.2.1, 1.2.2 Aug 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant