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

Implement new method for code request token generation #466

Merged
merged 3 commits into from
Sep 12, 2013

Conversation

pastoso
Copy link
Contributor

@pastoso pastoso commented Sep 12, 2013

It should be working now, although it would be nice to have more testing

@mofakfree
Copy link
Contributor

@pastoso thank you very much
but new function not work
Also The error appears
" [status] => fail
[reason] => bad_token"

@pastoso
Copy link
Contributor Author

pastoso commented Sep 12, 2013

@mofakfree can you tell me the phone number you are using for registering? I can try to figure out what's going wrong. Thanks!

@pastoso
Copy link
Contributor Author

pastoso commented Sep 12, 2013

@mofakfree can you try with the last commit? I though that curl was doing the urlencoding itself of the params, but it looks like when the base64 token had "/" it was failing

@mofakfree
Copy link
Contributor

@pastoso I try with this number 966550883606
Also you edit a previous amendment " 'token' => urlencode($token), "
but Still the same error appears .
[reason] => bad_token)
Can you send the message from you ? for test

shirioko added a commit that referenced this pull request Sep 12, 2013
Implement new method for code request token generation
@shirioko shirioko merged commit 7014234 into venomous0x:master Sep 12, 2013
@shirioko
Copy link
Contributor

@pastoso congratulations from Jake and me! :)

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

Successfully merging this pull request may close these issues.

None yet

3 participants