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

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

Merged

Conversation

fiddler
Copy link

@fiddler fiddler commented Jan 16, 2018

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!

@fiddler fiddler changed the title Separate user fetch strategies for normal and short password reset to… Fix reset password short reset token to work as advertised in the documentation Jan 16, 2018
@eddyystop eddyystop merged commit 8552086 into feathersjs-ecosystem:master Jan 25, 2018
@eddyystop
Copy link
Collaborator

Will be published as v2.x.x.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants