Skip to content

make the advantage and disadvantage of the ACL more clear #2989

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

Closed
wants to merge 3 commits into from
Closed

make the advantage and disadvantage of the ACL more clear #2989

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
365a257
make the advantage and disadvantage of the ACL more clear
Sep 18, 2013
b4c0c25
added xabbuh corrections
Sep 18, 2013
e8d8407
added stof´s corrections
Sep 18, 2013
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 13 additions & 0 deletions cookbook/security/acl.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -216,3 +216,16 @@ added above:
The user is now allowed to view, edit, delete, and un-delete objects.

.. _`MongoDBAclBundle`: https://github.com/IamPersistent/MongoDBAclBundle
Copy link
Member

Choose a reason for hiding this comment

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

external links should stay at the bottom of the file


Looking for alternatives
Copy link
Member

Choose a reason for hiding this comment

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

Alternatives

------------------------

For more simple use cases this seems to be an overhead for many developer.
Copy link
Member

Choose a reason for hiding this comment

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

developers

As a downside there is that in case you change the permissions in any way,
you need to change all the existing granted permissions as well. As well
when you want to change a specific part of the ACL you need to understand
the structure and process of this complex voter. So you should consider if
a classical and simple conditional permission check would fit for you.
Some do it with the existing voter structure as well, or custom build solutions.

.. _`Voters`: http://symfony.com/doc/current/cookbook/security/voters.html
Copy link
Member

Choose a reason for hiding this comment

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

links to other pages of the doc should not hardcode the url but use internal references instead (so that they link to the same version of the doc)