Skip to content
This repository has been archived by the owner. It is now read-only.

added /password/forgot/status endpoint #636

Merged
merged 1 commit into from Mar 20, 2014
Merged

Conversation

@dannycoates
Copy link
Member

@dannycoates dannycoates commented Mar 20, 2014

With this you can do a GET with your passwordForgotToken to get how many tries and the ttl remaining on the given token. When the token is consumed by a successful reset or expires you can expect to get a 401 HTTP status code with an errno of 110.

r?

@dannycoates
Copy link
Member Author

@dannycoates dannycoates commented Mar 20, 2014

fixes #635

@ckarlof
Copy link
Contributor

@ckarlof ckarlof commented Mar 20, 2014

r+ from a design perspective.

@zaach zaach mentioned this pull request Mar 20, 2014
2 of 2 tasks complete
@dannycoates dannycoates added the r? label Mar 20, 2014

:lock: HAWK-authenticated with the passwordForgotToken.

Returns the status for the passwordForgotToken. While the request returns a success response, the token has not yet been consumed. When the token is consumed by a successful reset or expires you can expect to get a 401 HTTP status code with an errno of 110.

This comment has been minimized.

@rfk

rfk Mar 20, 2014
Member

Ditto nit in #637, s/While/If/ would be clearer IMHPO

@rfk
Copy link
Member

@rfk rfk commented Mar 20, 2014

r+ with the same slight moral reservations as #637...

@chilts
Copy link
Contributor

@chilts chilts commented Mar 20, 2014

:squirrel:

dannycoates added a commit that referenced this pull request Mar 20, 2014
added /password/forgot/status endpoint
@dannycoates dannycoates merged commit a4403fd into mozilla:master Mar 20, 2014
@dannycoates dannycoates deleted the dannycoates:i635 branch Mar 20, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants