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

wasn't checking length in editNewPassword #80

Closed
wants to merge 1 commit into from

Conversation

blak3r
Copy link

@blak3r blak3r commented Jun 10, 2013

I had a similar issue this weekend with reset password. But, perhaps you just fixed that issue sometime after saturday because it seems correct now.

I had a similar issue this weekend with reset password.  But, perhaps you just fixed that issue sometime after saturday because it seems correct now.
@panique
Copy link
Owner

panique commented Jun 10, 2013

I just pushed a small fix [regarding this] into develop and directly merged it into master branch. Both are live here on github now. The editNewPassword() method had an error: it said $_POST['user_password_new'] >= 6, but strlen($_POST['user_password_new']) >= 6 would be right. Maybe i simply missed this or messed something up when experimenting with git. Anyway, fixed!!

@panique panique closed this Jun 10, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants