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 for ?scopes= in metadata server #333

Closed
wants to merge 3 commits into from
Closed

Support for ?scopes= in metadata server #333

wants to merge 3 commits into from

Conversation

asciimike
Copy link

Adds support for requesting access tokens with arbitrary metadata scopes from the compute metadata server.

Note: even though this feature is implemented in the metadata server, this feature is only available in the implementations on GAE Second Generation and GCF, not on GCE or GKE (where this functionality is already available by setting instance scopes).

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 30, 2019
@yoshi-automation yoshi-automation added the 🚨 This issue needs some love. label Apr 6, 2019
@yoshi-automation yoshi-automation removed the 🚨 This issue needs some love. label Jul 3, 2019
@sqrrrl
Copy link

sqrrrl commented Oct 21, 2019

ping

@@ -198,7 +201,7 @@ def get_service_account_token(request, service_account='default'):
"""
token_json = get(
request,
'instance/service-accounts/{0}/token'.format(service_account))
'instance/service-accounts/{0}/token?scopes={1}'.format(service_account, ",".join(scopes)))
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should URI encode values here.

@asciimike
Copy link
Author

I've since left Google, so I don't know what the internal status of this feature is or where the GCE or Client Lib team wants to go with it. Unless I get clear direction on that, I'll probably just close the PR and let them sort it out when they've prioritized it appropriately.

@busunkim96
Copy link
Contributor

Closing as this is superseded by #376

@busunkim96 busunkim96 closed this Jan 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants