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

Issue 765713 link

Starred by 1 user
Status: Fixed
Owner: ----
Closed: Sep 2017
Cc:
█ robert...@chromium.org
chanli@chromium.org
lijeffrey@chromium.org
liaoyuke@chromium.org
st...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

[Findit]OAuthServiceFailureError when getting oauth user email

Project Member Reported by chanli@chromium.org, Sep 15 2017 
Error report: https://pantheon.corp.google.com/errors/17236844458368744779?time=P30D&filter&project=findit-for-me

Failrue log:

at GetOauthUserEmail (/base/data/home/apps/s~findit-for-me/11683-609f390.404064357708153680/gae_libs/http/auth_util.py:67)
at ValidateOauthUserForNewAnalysis (/base/data/home/apps/s~findit-for-me/11683-609f390.404064357708153680/common/acl.py:46)
at _ValidateOauthUser (/base/data/home/apps/s~findit-for-me/11683-609f390.404064357708153680/findit_api.py:238)
at AnalyzeBuildFailures (/base/data/home/apps/s~findit-for-me/11683-609f390.404064357708153680/findit_api.py:734)
at decorated (/base/data/home/apps/s~findit-for-me/11683-609f390.404064357708153680/first_party/gae_ts_mon/config.py:279)
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 19 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/d69a09413f4e8760176cdc0bcaa0740b0b963f42

commit d69a09413f4e8760176cdc0bcaa0740b0b963f42
Author: Chan <chanli@chromium.org>
Date: Tue Sep 19 18:57:31 2017

[Findit] Handle OAuthRequestError in auth_util.

Bug:765713
Change-Id: I9e318da5358587340d0ad45ee15c47dfb717dbf3
Reviewed-on: https://chromium-review.googlesource.com/668687
Commit-Queue: Chan Li <chanli@chromium.org>
Reviewed-by: Shuotao Gao <stgao@chromium.org>
Reviewed-by: Jeffrey Li <lijeffrey@chromium.org>

[modify] https://crrev.com/d69a09413f4e8760176cdc0bcaa0740b0b963f42/appengine/findit/gae_libs/http/auth_util.py
[modify] https://crrev.com/d69a09413f4e8760176cdc0bcaa0740b0b963f42/appengine/findit/gae_libs/http/test/auth_util_test.py

Comment 2 by chanli@chromium.org, Sep 23 2017

Status: Fixed (was: Available)
►

Sign in to add a comment