Skip to content
This repository has been archived by the owner on Nov 5, 2019. It is now read-only.

Pass prompt='consent' from flow_from_clientsecrets #717

Merged
merged 2 commits into from
Jun 1, 2017

Conversation

tomasreimers
Copy link
Contributor

Currently there is no way to create an OAuth2WebServerFlow with a prompt argument using the flow_from_clientsecrets. Meaning developers must either call internal methods to add the prompt='consent' or read the client_secrets json themselves (so they can call the OAuth2WebServerFlow initializer themselves).

While this repository is technically deprecated, it is still pointed to by the official docs: https://developers.google.com/api-client-library/python/guide/aaa_oauth

Note: oauth2client is now deprecated. As such, it is unlikely that we will
review or merge to your pull request. We recommend you use
google-auth and oauthlib.

Currently there is no way to create an OAuth2WebServerFlow with a prompt argument using the flow_from_clientsecrets. Meaning developers must either call internal methods to add the prompt='consent' or read the client_secrets json themselves (so they can call the OAuth2WebServerFlow initializer themselves).
@theacodes
Copy link
Contributor

Seems like lint is failing on this PR. Can you address that?

@tomasreimers
Copy link
Contributor Author

@jonparrott good call and done!

@tomasreimers
Copy link
Contributor Author

@jonparrott travis is still failing, but I think it has to do with the tests and not my change... any advice?

@theacodes
Copy link
Contributor

@tomasreimers it's just the system tests which will not pass on external PRs.

@theacodes theacodes merged commit a1da72d into googleapis:master Jun 1, 2017
@theacodes
Copy link
Contributor

Thanks, @tomasreimers.

@tomasreimers
Copy link
Contributor Author

tomasreimers commented Jun 1, 2017

Thanks @jonparrott ! Any idea when you'll bump the version on pip?

@theacodes
Copy link
Contributor

If you'll send a release PR it'll bump as soon as I merge.

@tomasreimers
Copy link
Contributor Author

tomasreimers commented Jun 5, 2017 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants