Fix reset password short reset token to work as advertised in the documentation

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

Fix reset password short reset token to work as advertised in the documentation #88

Closed fiddler closed 6 years ago

fiddler commented 6 years ago

This PR reverts the library to work as previously with short reset tokens before PR #68. Meaning that short tokens don't get prepended with a user ID as that makes them not usable for their purpose. Also, to support this the method for fetching the user when resetting the password is reverted back to querying with a passed in user identity in case of the short token.

Changed the tests to reflect the short token not having ID prepended.

Check it out and let me know any improvements!

eddyystop commented 6 years ago

Will be published as v2.x.x.