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 #746 #748

Merged
merged 1 commit into from
May 1, 2015
Merged

Fix #746 #748

merged 1 commit into from
May 1, 2015

Conversation

aesedepece
Copy link
Contributor

Updates S40 and Android tokens and user agents.

@assegaf
Copy link

assegaf commented May 1, 2015

hi @aesedepece
I tried your changes on AndroidYowsupEnv

./yowsup-cli registration -p XXXXXXXXXXX -r sms --cc XX

INFO:yowsup.common.http.warequest:{"status":"fail","reason":"incorrect","sms_length":6,"voice_length":6}
INFO:yowsup.common.http.warequest:{"status":"fail","reason":"bad_token"}

status: fail
reason: bad_token

but the S40Environment seem working fine,

INFO:yowsup.common.http.warequest:{"status":"sent","length":6,"method":"sms","retry_after":1805}

any idea?

@aesedepece
Copy link
Contributor Author

@assegaf since long time ago, registration works better with S40's token and general use with Android's one. At least that's been my experience.

@assegaf
Copy link

assegaf commented May 1, 2015

@aesedepece
thanks, I wonder also, currently android max version is 2.12.67 (beta), but its written 2.12.68 (AndroidEnvironment)

do you need to requestexist new password on using Android Agent ? or just use its password got from S40 Registration directly ?

@mgp25
Copy link
Contributor

mgp25 commented May 1, 2015

@aesedepece
Copy link
Contributor Author

@assegaf WAUTH passwords work independently of the token you use.

@assegaf
Copy link

assegaf commented May 1, 2015

its working 2.12.67,

big thanks
@aesedepece
@mgp25

@lcmeyer37 lcmeyer37 mentioned this pull request May 1, 2015
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.

4 participants