Skip to content
This repository has been archived by the owner on Apr 3, 2019. It is now read-only.

POST /v1/account/unlock/verify_code throws 'contains invalid parameters' instead of ' invalid verification code' #937

Closed
vladikoff opened this issue May 19, 2015 · 1 comment
Closed

POST /v1/account/unlock/verify_code throws 'contains invalid parameters' instead of ' invalid verification code' #937

vladikoff opened this issue May 19, 2015 · 1 comment

Comments

@vladikoff
Copy link
Contributor

STR

Use the provided curl example in the docs against the local server (127.0.0.1:9000):
https://github.com/mozilla/fxa-auth-server/blob/master/docs/api.md#request-8

Expected

Either 200 OK {} OR status code 400, errno 105: invalid verification code

Actual

{"code":400,"errno":107,"error":"Bad Request","message":"Invalid parameter in request body","info":"https://github.com/mozilla/fxa-auth-server/blob/master/docs/api.md#response-format","validation":{"source":"payload","keys":["code"]},"log":[]}%

Details

Tested on fxa-auth-server ec669ceb37f754335e296d13ec985deefc2d122d
ref mozilla/fxa-js-client#151
@dannycoates
Copy link
Contributor

The example code should be 32 chars not 64.

@vladikoff vladikoff self-assigned this May 19, 2015
@rfk rfk added this to the train-39 milestone Jun 3, 2015
@vladikoff vladikoff modified the milestones: train-40, train-39 Jun 9, 2015
vladikoff added a commit to vladikoff/fxa-auth-server that referenced this issue Jun 9, 2015
@vladikoff
fix(docs): update documentation for example verification code, from 6…
5c3bf0b 
…4 to 32 chars

Fixes mozilla#937
[skip ci]
@vladikoff vladikoff mentioned this issue Jun 9, 2015
Merged
@vladikoff vladikoff removed their assignment Jun 9, 2015
@rfk rfk closed this as completed in #956 Jun 17, 2015
@vladikoff vladikoff mentioned this issue Aug 5, 2015
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@rfk @dannycoates @vladikoff