New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 620838 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Feature



Sign in to add a comment

Make Chromium Rietveld support oauth for App Engine apps

Project Member Reported by st...@chromium.org, Jun 16 2016

Issue description

For App Engine apps to authenticate itself to other apps through oauth is using the following code:

auth_token, _ = app_identity.get_access_token(email_scope)
headers['Authorization'] = 'Bearer ' + auth_token

However, in Chromium Rietveld, this won't work, because it only supports client-id-based oauth with oauth.get_client_id(email_scope). And for app engine requests with the above header, it just returns 'anonymous'.

We could support per-email oauth with oauth.get_current_user(email_scope).email(), as it returns the App Engine account (APP_ID@appspot.gserviceaccount.com) for the above header.
 

Comment 2 by st...@chromium.org, Jun 16 2016

Status: Fixed (was: Assigned)

Sign in to add a comment