WickedShell / echelon

http://echelon.bigbrotherbot.net
14 stars 7 forks source link

Password reset #41

Closed FrikFry closed 13 years ago

FrikFry commented 13 years ago

It doesn't appear to be working. I get the email and on the surface echelon goes through all the steps and even sends me back to the login page with the message that my password has been changed, but it doesn't actually change the password. Any thoughts?

WickedShell commented 13 years ago

Wow, alright this one was frustratingly stupid. Someone hadn't understood that a string is typecasted to a boolean. (and if there's anything in the string it's guranteed to type cast to true).

This should be fixed. It looks like the password reset function was working, but the password validty checker could fail, and it would return the success message.