-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Adding ability to sign URL from GAE. #921
Conversation
@jgeewax I was actually reminded of this old bug when discussing some performance issues of Is there a GCE equivalent (I assume via metadata server) of from google.appengine.api import app_identity
_, signed_bytes = app_identity.sign_blob(string_to_sign) Is there someone who works on GCE who might be worth asking? |
@ludoch : Do you know if there's a GCE equivalent of GAE Python's |
@tseaver PTAL |
if _GAECreds is not None and isinstance(credentials, _GAECreds): | ||
_, signed_bytes = app_identity.sign_blob(string_to_sign) | ||
return signed_bytes | ||
else: |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
Also refactoring _get_signed_query_params and the related tests so that the signing process and service account name determination are isolated methods. Fixes googleapis#607.
@tseaver Done (and rebased on top of HEAD). |
LGTM |
Adding ability to sign URL from GAE.
Also refactoring
_get_signed_query_params
and the related tests so that the signing process and service account name determination are isolated methods.Fixes #607.