search

feathersjs-ecosystem / feathers-authentication-management

Adds sign up verification, forgotten password reset, and other capabilities to local feathers-authentication
https://feathers-a-m.netlify.app/
MIT License
246 stars 98 forks source link

Use the passwordField config for hashing the password #115

Closed osinga closed 6 years ago

osinga commented 6 years ago

As described in #64, when the authentication.local.passwordField config is set to anything other than the default password the hashPassword helper will not function properly. Consequently, the resetToken is being saved unhashed, preventing you from using the resetPwdLong action successfully.

eddyystop commented 6 years ago

All the 16 sendResetPwd tests are failing.

I'm starting a rewrite of this repo in early October, converting it to async/await, and adding features that did not make it into the present rewrite.

All the outstanding issues will be addressed at that time.

osinga commented 6 years ago

Odd. They're not failing on my side. Close it for now then, awaiting the rewrite?

eddyystop commented 6 years ago

I would suggest yes.