vinitkumar / googlecl

GoogleCL rewrite in __progress__
Other
325 stars 48 forks source link

Add support for @googlemail.com users #224

Closed vinitkumar closed 10 years ago

vinitkumar commented 10 years ago

From cmsjtenshu on July 08, 2010 19:04:35

What steps will reproduce the problem? 1. Have an @googlemail.com account

  1. Try to auth googlecl
  2. There is no step three What is the expected output? What do you see instead? I would expect to end up with a valid OAuth session. Instead I see:

Please specify user: mykooluzar Please log in and/or grant access via your browser at https://www.google.com/accounts/OAuthAuthorizeToken?oauth_token=blahblah&hd=default then hit enter. You specified account mykooluzar but granted access for mykooluzr@googlemail.com Please log out of mykooluzar@googlemail.com and grant access with mykooluzar

(it doesn't work if I specify a user of mykooluzar@googlemail.com, that just says my domain isn't using Apps For Domain. Also 'mykooluzar' isn't my real username obviously ;) What version of the product are you using? On what operating system? What version of gdata-python-client (aka python-gdata)? Tried with lucid's version and 2.0.8 from sid Please provide any additional information below.

Original issue: http://code.google.com/p/googlecl/issues/detail?id=216

vinitkumar commented 10 years ago

From tom.h.mi...@gmail.com on July 08, 2010 08:30:18

I suspect this patch will fix the problem. Could you let me know how it works?

Attachment: googlemail.patch

vinitkumar commented 10 years ago

From cmsjtenshu on July 08, 2010 08:33:56

Doh, I literally just used the rebrand tool to turn my googlemail.com account into a gmail.com one :/

vinitkumar commented 10 years ago

From tom.h.mi...@gmail.com on July 08, 2010 09:30:52

Oh well. I'm going to guess that the patch works -- it certainly won't mess anything up for current users.

Thanks anyway!

Status: Fixed

vinitkumar commented 10 years ago

From tom.h.mi...@gmail.com on July 08, 2010 09:38:17

committed as r332

vinitkumar commented 10 years ago

From tom.h.mi...@gmail.com on July 11, 2010 09:09:34

Issue 221 has been merged into this issue.