Skip to content

[Security] Missing documentation regarding AccessDeniedHandlerInterface #5139

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.

Sign up for GitHub

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

Jump to bottom
Closed
linaori opened this issue Apr 3, 2015 · 1 comment
Closed

[Security] Missing documentation regarding AccessDeniedHandlerInterface #5139

linaori opened this issue Apr 3, 2015 · 1 comment
Labels
actionable Clear and specific issues ready for anyone to take them. good first issue Ideal for your first contribution! (some Symfony experience may be required) hasPR A Pull Request has already been submitted for this issue. Security

Comments

@linaori
Copy link
Contributor

linaori commented Apr 3, 2015

I've seen a few occasion where custom access denied handling is required. It's not rocket science to implement, you put access_denied_handler: service.id on your firewall configuration and implement the AccessDeniedHandlerInterface in this service.

I think this could be a simple explanation but add some real value to the docs.
@xabbuh xabbuh added Doc Request actionable Clear and specific issues ready for anyone to take them. Security labels Apr 3, 2015
@wouterj
Copy link
Member

wouterj commented May 10, 2015

I vote to add this information to reference/configuration/framework

@wouterj wouterj added the good first issue Ideal for your first contribution! (some Symfony experience may be required) label May 10, 2015
@linaori linaori mentioned this issue Apr 19, 2016
Closed
@nykopol nykopol mentioned this issue Oct 31, 2016
Closed
@xabbuh xabbuh added the hasPR A Pull Request has already been submitted for this issue. label Nov 2, 2016
xabbuh added a commit that referenced this issue Nov 28, 2016
@xabbuh
feature #7105 Add documentation about access denied handler (nykopol,…
39481b3 
… javiereguiluz)

This PR was submitted for the master branch but it was merged into the 2.7 branch instead (closes #7105).

Discussion
----------

Add documentation about access denied handler

Fix #5139

Commits
-------

ea4b7f8 Minor rewordings
dff303c When talking about generic responses, don't spell it as Response (which is a Symfony class)
a9f815a xabbuh review
66c0e76 fix typo
db65dd8 fix yml marker to yaml
a17965e add documentation about access denied handler
@xabbuh xabbuh closed this as completed Nov 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
actionable Clear and specific issues ready for anyone to take them. good first issue Ideal for your first contribution! (some Symfony experience may be required) hasPR A Pull Request has already been submitted for this issue. Security
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@wouterj @linaori @xabbuh