Skip to content

[Security] Firewall context stateless tip #7827

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
Apr 27, 2017

Conversation

geoffrey-brier
Copy link
Contributor

I recently faced this situation.
Only took me a few minutes to find out why but IMO it's worth adding a note ;)

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.

👍

@javiereguiluz
Copy link
Member

Thinking about this help note, don't you think this is a Symfony bug? If the "context" option only works when doing some things, shouldn't Symfony check if you have done those things and warn you if you haven't?

@geoffrey-brier
Copy link
Contributor Author

@javiereguiluz Not sure if it's a bug but that may be a good idea to check that.

@xabbuh
Copy link
Member

xabbuh commented Apr 27, 2017

Thank you @geoffrey-brier.

@xabbuh xabbuh merged commit 9533b68 into symfony:2.7 Apr 27, 2017
xabbuh added a commit that referenced this pull request Apr 27, 2017
…, javiereguiluz)

This PR was merged into the 2.7 branch.

Discussion
----------

[Security] Firewall context stateless tip

I recently faced this situation.
Only took me a few minutes to find out why but IMO it's worth adding a note ;)

Commits
-------

9533b68 Minor reword and reformatting
64f1fa0 Add security stateless tip
@geoffrey-brier geoffrey-brier deleted the firewall-context-stateless-tip branch April 27, 2017 21:18
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