Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[Security] Update description of password hasher config#15430

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
javiereguiluz merged 1 commit intosymfony:5.3frompableu:security-hashers
Jun 11, 2021

Conversation

@pableu
Copy link
Contributor

The description of the password hashers in the reference isn't up to date for Symfony 5.3.

"Auto" now always uses bcrypt (see#14980 and#14992), but it wasn't reflected here. I initially thought this was a bug in the password hasher component itself and created asymfony/symfony#41646, but I've since learned that the switch to bcrypt was intentional :-)

I updated all the hasher descriptions a bit and removed the part about sodium before PHP 7.2 because Symfony 5.3 requires PHP >= 7.2. I also added an extra paragraph for the bcrypt hasher because it was a bit mixed into the description of the "auto" hasher.

Copy link
Member

@chalasrchalasr left a comment

Choose a reason for hiding this comment

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

Thanks!

pableu reacted with rocket emoji
@javiereguiluzjaviereguiluz added this to the5.3 milestoneJun 11, 2021
@javiereguiluzjaviereguiluz merged commit74e53fb intosymfony:5.3Jun 11, 2021
@javiereguiluz
Copy link
Member

Thanks Pablo!

pableu reacted with hooray emoji

@pableupableu deleted the security-hashers branchOctober 4, 2021 08:07
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

1 more reviewer

@chalasrchalasrchalasr approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Projects

None yet

Milestone

5.3

Development

Successfully merging this pull request may close these issues.

4 participants

@pableu@javiereguiluz@chalasr@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp