-
Notifications
You must be signed in to change notification settings - Fork 31
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
Add support for logging in via an OAuth token #41
Conversation
This method is unfinished, bcz if gmail havent recovery mail or phone number then you cant access to gmail. Have to add recovery mail to confirm it then you will get token from cookie. I did it in my php code and working good. I will fix python then pull it if you interest to use python/gpsoauth. |
I create this function and works perfectly. But I have to obtain the OAuth before with selenium.
|
@emma-eva can you provide your contact info I am also working on PHP code but getting a few issues. |
I think we're good to keep the existing interface with the changes from #42, so I'm going to close this for now. |
@kiwiz I don't have the bandwidth to take this change over but I can give feedback on the PR if you or someone else wants to work on it. I think the main things are a) setting up docs for the flow and b) figuring out the interface. Maybe a separate method like exchange_token? |
@simon-weber I've split out the logic into a separate function and added some docs to the README. There's a more thorough description here that I could reference/link if appropriate. |
@simon-weber Feedback on this PR would be much appreciated! |
I have tested it and it actually works pretty good. Using it right now. |
Sorry for the delay -- I was traveling the past couple of weeks. This looks good; thanks! |
All good! Thanks for your work maintaining this library. :] |
Relates to #37.
It appears that Google made some changes recently which breaks the standard "master login" flow. In order to fetch a token, the user has to:
oauth_token
cookie that is populategpsooauth
Necessary changes determined from https://gitlab.com/AuroraOSS/AuroraStore/-/blob/master/app/src/main/java/com/aurora/store/util/AC2DMTask.kt#L27-43 and tested locally.
Here's some example code, if useful for docs: