Recover Module Should Use UpdatePassword Method

volatiletech / authboss

The boss of http auth.

MIT License

3.81k stars 207 forks source link

Recover Module Should Use UpdatePassword Method #308

Open ibraheemdev opened 4 years ago

ibraheemdev commented 4 years ago

Shouldn't the EndPost() method in the recover module use the UpdatePassword() method, instead of essentially rewriting it? Currently UpdatePassword() is defined but not used internally. Should I make a PR, or is there a reason for this?

aarondl commented 4 years ago

Doesn't seem that there's any reason other than EndPost was written before UpdatePassword was. A PR would be accepted.