Skip to content
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

[6.x] Ability to override `set user's password` on password reset #30218

Merged
merged 1 commit into from Oct 9, 2019

Conversation

@arcanedev-maroc
Copy link
Contributor

arcanedev-maroc commented Oct 8, 2019

This PR solves the issue of hashing the password twice when using a mutator like setPasswordAttribute in User model.

Also, avoiding to override the whole method in Illuminate\Foundation\Auth\ResetsPasswords to solve this issue.

Related: #11012

This PR solves the issue of hashing the password twice when using a mutator like `setPasswordAttribute` in User model
@taylorotwell taylorotwell merged commit 6e845d7 into laravel:6.x Oct 9, 2019
2 checks passed
2 checks passed
continuous-integration/styleci/pr The analysis has passed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@arcanedev-maroc arcanedev-maroc deleted the ARCANEFORKS:patch-1 branch Oct 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.