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

Option to Disable Automatic Webbrowser Opening #57

Merged
merged 1 commit into from
Oct 1, 2020

Conversation

uberfastman
Copy link
Contributor

Currently the OAuth process in the yahoo-oauth library always opens a web browser when a new token is requested. However, if running OAuth inside a container (Docker), it is extremely difficult (if not impossible in some environments) for the Docker container to talk to the host machine.

As such, this small PR adds the ability to include a keyword/named argument, browser_callback, which, when set to False, will simply include the authorization URL in the verification code input request, allowing users to click on the link themselves to get the Yahoo OAuth verification code.

@josuebrunel
Copy link
Owner

Hello @uberfastman ,
Thanks for the PR, and I understand the need. I'll do my best to review it as soon as possible.
I keep you posted

@uberfastman
Copy link
Contributor Author

Thanks so much @josuebrunel!

@josuebrunel josuebrunel merged commit a1ec871 into josuebrunel:master Oct 1, 2020
@uberfastman
Copy link
Contributor Author

@josuebrunel thanks so much for accepting my PR! I would be grateful as well if you could let me know when the PyPi package has been updated so I can pull the latest version that includes these updates!

@josuebrunel
Copy link
Owner

@uberfastman it's on pypi now. Let me know if it works as expected

@uberfastman
Copy link
Contributor Author

@josuebrunel works perfectly, thanks so much!

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.

2 participants