Skip to content

Info about CSRF protection #5663

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

Closed
wants to merge 1 commit into from
Closed

Conversation

ThomasLandauer
Copy link
Contributor

This should be explained somewhere (took me a while to figure it out). If this is not the right place (or format), please go ahead and correct it!

This should be explained *somewhere* (took me a while to figure it out). If this is not the right place (or format), please go ahead and correct it!
@xabbuh
Copy link
Member

xabbuh commented Sep 1, 2015

I would not explain this behaviour in the HttpFoundation component docs. The components section is meant to explain how to use components standalone. Thus, your changes are better suited in the book (like you did in #5664). Though we can check if we are in the need to add something like this to the Form component section.

@ThomasLandauer
Copy link
Contributor Author

Since there is no chapter about session in the book (BTW: Why not?), this is the only session-related docs there are out there. And (as I said in the other Pull Request), the interplay between sessions and forms should be explained somewhere. Of course, I don't insist that here is the right place ;-)

Cheers,
Thomas

@wouterj
Copy link
Member

wouterj commented Feb 6, 2016

With the note you added in #5664, I don't think we need anything more. For that reason, I'm going to close this PR for now. Feel free to comment if you don't agree.

@wouterj wouterj closed this Feb 6, 2016
@ThomasLandauer ThomasLandauer deleted the patch-1 branch August 17, 2018 09:40
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.

3 participants