search

iitc-project / ingress-intel-total-conversion

ingress.com/intel total conversion user script with some new features. Should allow easier extension of the intel map.
http://iitc.jonatkins.com/
ISC License
989 stars 552 forks source link

Could not Login in mobile iitc since 0.3 version #221

Closed Guria closed 11 years ago

Guria commented 11 years ago

Mobile app opens login page in external browser. After login and grant permissions it redirects back to mobile app, but app seems doesn't gets access. Also cheched in test build.

jonatkins commented 11 years ago

@leCradle I seem to remember some discussion on this issue, where we may need to support other URLs for login?

@Guria can you confirm the URL that's opened in the external browser?

leCradle commented 11 years ago

@Guria

Please post the url you were redirected to... This is a known issue but I always missed to capture this url...because it asks for access only 1 time in 30 days... Force close the app and login again should temporary fix this issue...

Guria commented 11 years ago

Of course. It doesn't happens if user earlier marked checkbox on grant permissions page and doesn't enabled 2-step authorisation. For first case url is: https://appengine.google.com/_ah/loginform?state=

leCradle commented 11 years ago

@Guria Thx! I think it should be enough to start the appengine page inside iitcm...otherwise the cookies are saved at the wrong place. If my theory is correct, this patch should fix it http://p.nnev.de/3581 Unfortunately, I am not at my workstation and will not be until tomorrow evening. Eventually @jonatkins can build a test.apk including this patch for you...?

jonatkins commented 11 years ago

new test build available at http://iitc.jonatkins.com/?page=test#test-mobile

leCradle commented 11 years ago

@Guria does it works for you? If it works, we can close this issue...

jonatkins commented 11 years ago

I had this two-stage login process tonight - the user/pass prompt, followed by the 'which account' prompt - and it worked fine. This is the thing that wasn't working correctly, yes?

leCradle commented 11 years ago

@jonatkins if I understood @Guria correctly, then yes... At least that was the issue I patched... :D

Guria commented 11 years ago

Thanks.