Skip to content

Added possible values for access_decision_manager.strategy #6464

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 1 commit into from
Apr 21, 2016

Conversation

a-ast
Copy link
Contributor

@a-ast a-ast commented Apr 12, 2016

Q A
Doc fix? yes
New docs? no
Applies to 2.3+
Fixed tickets n/a

Added a comment with the list of possible values for access_decision_manager.strategy (like for session_fixation_strategy)

Added a comment with the list of possible values for access_decision_manager.strategy (like for session_fixation_strategy)
@xabbuh
Copy link
Member

xabbuh commented Apr 19, 2016

👍 (even if this isn't part of the output of the debug:config command, it's still useful for readers)

@ogizanagi
Copy link
Contributor

ogizanagi commented Apr 19, 2016

@xabbuh, @AAstakhov : Maybe this should conform the config:dump-reference security output then:

    access_decision_manager:
        strategy:             affirmative # One of "affirmative"; "consensus"; "unanimous"
        allow_if_all_abstain:  false
        allow_if_equal_granted_denied:  true

@xabbuh
Copy link
Member

xabbuh commented Apr 19, 2016

@ogizanagi Indeed, good idea

@weaverryan
Copy link
Member

weaverryan commented Apr 21, 2016

Thanks Andrey! I'll make the small change suggested

@weaverryan weaverryan merged commit 73d96d4 into symfony:2.3 Apr 21, 2016
weaverryan added a commit that referenced this pull request Apr 21, 2016
…y (AAstakhov)

This PR was merged into the 2.3 branch.

Discussion
----------

Added possible values for access_decision_manager.strategy

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | 2.3+
| Fixed tickets | n/a

Added a comment with the list of possible values for access_decision_manager.strategy (like for session_fixation_strategy)

Commits
-------

73d96d4 Added possible values for access_decision_manager.strategy
weaverryan added a commit that referenced this pull request Apr 21, 2016
xabbuh added a commit that referenced this pull request Apr 24, 2016
* 2.3:
  remove documentation of not supported "verbose" option value
  use port 587 in Amazon SES example
  [#6464] Minor tweak
  Fix one more occurence of /read
  Fix typo
  Added possible values for access_decision_manager.strategy
xabbuh added a commit that referenced this pull request Apr 24, 2016
* 2.7:
  remove documentation of not supported "verbose" option value
  use port 587 in Amazon SES example
  [#6464] Minor tweak
  Fix one more occurence of /read
  Fix typo
  Added possible values for access_decision_manager.strategy
xabbuh added a commit that referenced this pull request Apr 24, 2016
* 2.8:
  [#6465] fix newline position
  remove documentation of not supported "verbose" option value
  Update http_kernel_httpkernel_class.rst
  use port 587 in Amazon SES example
  fix typo
  [#6464] Minor tweak
  Fix one more occurence of /read
  Fix typo
  Added ldap to the list of user providers
  Added possible values for access_decision_manager.strategy
xabbuh added a commit that referenced this pull request Apr 24, 2016
* 3.0:
  [#6465] fix newline position
  remove documentation of not supported "verbose" option value
  Update http_kernel_httpkernel_class.rst
  use port 587 in Amazon SES example
  fix typo
  [#6464] Minor tweak
  Fix one more occurence of /read
  Fix typo
  Added ldap to the list of user providers
  Added possible values for access_decision_manager.strategy
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