Skip to content

Conversation

@dhermes
Copy link
Contributor

@dhermes dhermes commented Feb 20, 2016

@tseaver

  1. Maybe this isn't the best way to solve oauthclient.OAuth2Credentials does not implement create_scoped or create_scoped_required #1412? Happy to hear suggestions.
  2. I also realized that we don't really need the get_for_service_account_p12 and get_for_service_account_json helpers any more since the ServiceAccountCredentials factories do everything in a nice short way.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 20, 2016
@tseaver
Copy link
Contributor

tseaver commented Feb 20, 2016

LGTM

dhermes added a commit that referenced this pull request Feb 20, 2016
Allowing credentials that don't implement create_scoped().
@dhermes dhermes merged commit 252d532 into googleapis:master Feb 20, 2016
@dhermes dhermes deleted the fix-1412 branch February 20, 2016 02:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: core auth cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants