Skip to content

Update http_cache.rst #8442

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

Merged
merged 2 commits into from
Oct 4, 2017
Merged

Update http_cache.rst #8442

merged 2 commits into from
Oct 4, 2017

Conversation

tabbi89
Copy link
Contributor

@tabbi89 tabbi89 commented Oct 2, 2017

I want to point out that POST method is cacheable however this is very very narrow case but it is possible. Reading documentation you can learn that POST is not cacheable but this is not true and technically you can cache it.

I want to point that POST method is cacheable however this is very very narrow case but it is possible. Reading documentation you can learn that POST is not cacheable but this is not true and technically you can cache it.
Copy link
Member

@javiereguiluz javiereguiluz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tabbi89 thanks a lot for this! I wasn't aware of this at all.

I've reworded a bit your pull request to expand the explanation and to link to the HTTP spec.

@tabbi89
Copy link
Contributor Author

tabbi89 commented Oct 3, 2017

@javiereguiluz looks great :)

@wouterj
Copy link
Member

wouterj commented Oct 4, 2017

Thanks, looks great!

@wouterj wouterj merged commit e89aab0 into symfony:2.7 Oct 4, 2017
wouterj added a commit that referenced this pull request Oct 4, 2017
This PR was merged into the 2.7 branch.

Discussion
----------

Update http_cache.rst

I want to point out that POST method is cacheable however this is very very narrow case but it is possible. Reading documentation you can learn that POST is not cacheable but this is not true and technically you can cache it.

Commits
-------

e89aab0 Reworded the explanation abut POST caching
44e569f Update http_cache.rst
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants