Skip to content

Commit 3d4b22f

Browse files
committed
minor #9363 Clarify unanimous access strategy description (Steveb-p, javiereguiluz)
This PR was submitted for the 4.0 branch but it was merged into the 2.7 branch instead (closes #9363). Discussion ---------- Clarify unanimous access strategy description In my opinion saying that unanimous strategy "requires all voters to grant access" is misleading. In actuality, this strategy requires that no voter denies access, and this PR intends to clarify that. Commits ------- b4c2e90 Minor reword 94d6f30 Clarify unanimous access strategy description
2 parents fa65333 + b4c2e90 commit 3d4b22f

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

security/voters.rst

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -251,7 +251,9 @@ strategies available:
251251
This grants access if there are more voters granting access than denying;
252252

253253
``unanimous``
254-
This only grants access once *all* voters grant access.
254+
This only grants access if there is no voter denying access. If all voters
255+
abstained from voting, the decision is based on the ``allow_if_all_abstain``
256+
config option (which defaults to ``false``).
255257

256258
In the above scenario, both voters should grant access in order to grant access
257259
to the user to read the post. In this case, the default strategy is no longer

0 commit comments

Comments
 (0)