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

Cannot get resetPasswordUrl from user.forgotPassword #146

Closed
me717 opened this issue Jan 17, 2018 · 3 comments
Closed

Cannot get resetPasswordUrl from user.forgotPassword #146

me717 opened this issue Jan 17, 2018 · 3 comments

Comments

@me717
Copy link

me717 commented Jan 17, 2018

The ForgotPasswordResponse object returned by forgotPassword() and forgotPassword(sendEmail, userCredentials) method of the user class never has the resetPasswordUrl property initialized.

This appears to be because the backend api only return that field if sendEmail is false, but no user credentials are included in the request (as described here). This is a call that is presently impossible to make from this sdk. @bdemers Would it be possible to get a forgotPassword(sendEmail) method, or be able to include null credentials?

@bdemers
Copy link
Contributor

bdemers commented Jan 17, 2018

For a work around try calling user.forgotPassword(false, client.instantiate(UserCredentials.class)) this should make the request with an empty payload.

@me717
Copy link
Author

me717 commented Jan 17, 2018

That results in an error with the following causes: password: The field cannot be left blank;recoveryQuestion: The field cannot be left blank

@bdemers
Copy link
Contributor

bdemers commented Jan 18, 2018

@me717 OK we will look into that. Thanks for trying it out and getting back to us!

bdemers added a commit that referenced this issue Jan 18, 2018
`User.forgotPassword()` has an optional arg `UserCredentials`.  If this argument can now be null.
Included IT

Fixes: #146
bdemers added a commit that referenced this issue Jan 31, 2018
`User.forgotPassword()` has an optional arg `UserCredentials`.  If this argument can now be null.
Included IT

Fixes: #146
bdemers added a commit that referenced this issue Jan 31, 2018
`User.forgotPassword()` has an optional arg `UserCredentials`.  If this argument can now be null.
Included IT

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

No branches or pull requests

2 participants