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

Support non-service-account credential files when run as a binary #217

Merged
merged 1 commit into from
Oct 1, 2018

Conversation

hfwang
Copy link
Contributor

@hfwang hfwang commented Sep 28, 2018

For example, gcloud's credential files will have either type authorized_user or service_account. JWTConfigFromJSON only supports service accounts, while CredentialsFromJSON supports both service accounts and authorized users.

This was tested with a regular gmail account and a service account using the legacy JSON auth files created by gcloud in .config/gcloud/legacy_credentials/*/adc.json

…nary.

For example, gcloud's credential files will have either type authorized_user or service_account. JWTConfigFromJSON only supports service accounts, while CredentialsFromJSON supports both service accounts and authorized users.

This was tested with a regular gmail account and a service account using the legacy JSON auth files created by gcloud in .config/gcloud/legacy_credentials/*/adc.json
@hfwang hfwang requested a review from easwars September 28, 2018 19:35
@hfwang hfwang added priority: p3 Desirable enhancement or fix. May not be included in next release. Status: Review Needed type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. labels Sep 28, 2018
@hfwang hfwang merged commit 3236ed5 into GoogleCloudPlatform:master Oct 1, 2018
@hfwang hfwang deleted the authorized_user branch October 1, 2018 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: p3 Desirable enhancement or fix. May not be included in next release. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants