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

update_with_password does not check to see if new password and confirmation are the same #1947

Closed
mdoroudi opened this issue Jun 25, 2012 · 5 comments

Comments

@mdoroudi
Copy link

update_with_password doesn't check if the new password and confirmation matches so it just accepts the new password regardless.
I'm already working on it so I can just submit the fix.

@josevalim
Copy link
Contributor

the validation will only happen if you have validatable included, it is not part of update_with_password to validate new password and friends.

@joelcdoyle
Copy link

This is the exact issue I am having. Even if validatable is included, update_with_password does not return false if password and password_confirmation are different or if password_confirmation is nil.

@joelcdoyle
Copy link

@mdoroudi I would like to see your fix.

@josevalim
Copy link
Contributor

@joelcdoyle I see now. I will reopen the other issue, it is indeed a bug.

@joelcdoyle
Copy link

Yey! I helped. Thanks, @josevalim

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants