10000 [Security] Update security.rst by mark2016 · Pull Request #16576 · symfony/symfony-docs · GitHub
[go: up one dir, main page]
Skip to content

[Security] Update security.rst #16576

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
Merged
merged 1 commit into from
Mar 11, 2022
Merged

[Security] Update security.rst #16576

merged 1 commit into from
Mar 11, 2022

Conversation

mark2016
Copy link
Contributor
@mark2016 mark2016 commented Mar 7, 2022

Added the config value for failure_path under form_login (it was missing).

94noni
94noni approved these changes Mar 7, 2022
View reviewed changes
Copy link
Contributor
@94noni 94noni left a comment

Choose a reason for hiding this comment

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

shouldn't this be documented for v4 and/or v5 as well?

@javiereguiluz javiereguiluz added this to the 4.4 milestone Mar 11, 2022
@carsonbot carsonbot changed the title Update security.rst [Security] Update security.rst Mar 11, 2022
@javiereguiluz javiereguiluz changed the base branch from 6.0 to 4.4 March 11, 2022 11:04
@mark2016 @javiereguiluz
Update security.rst
fde7ca7 
Added the config value for failure_path under form_login (it was missing).
@javiereguiluz javiereguiluz requested a review from xabbuh as a code owner March 11, 2022 11:04
@javiereguiluz javiereguiluz merged commit b309307 into symfony:4.4 Mar 11, 2022
@javiereguiluz
Copy link
Member

Thanks @mark2016 !

@94noni we merged it in 4.4 branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@94noni 94noni 94noni approved these changes

@xabbuh xabbuh Awaiting requested review from xabbuh

Assignees
No one assigned
Labels
Security Status: Needs Review
Projects
None yet
Milestone
4.4
Development

Successfully merging this pull request may close these issues.
4 participants
@mark2016 @javiereguiluz @94noni @carsonbot
0