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

Issue 805220 link

Starred by 1 user
Status: Fixed
Owner:
█ robert...@chromium.org
Last visit > 30 days ago
Closed: Jan 2018
Cc:
█ robert...@chromium.org
chanli@chromium.org
lijeffrey@chromium.org
liaoyuke@chromium.org
st...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

ObscureEmails cron job fails with a 500 when an email is None

Project Member Reported by robert...@chromium.org, Jan 24 2018
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 24 2018 

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

commit 1c48cc219cfdd70c0dfc548e8ce5893eea0097ec
Author: Roberto Carrillo <robertocn@google.com>
Date: Wed Jan 24 01:42:38 2018

[Findit] Do not split None when obscuring emails.

This is causing 500s on the cron job.

R=chanli,wylieb,lijeffrey,stgao

Bug:  805220 
Change-Id: Ibc0eb46ca9fd5c1ddef82fcf528f004adab6fd90
Reviewed-on: https://chromium-review.googlesource.com/882504
Reviewed-by: Shuotao Gao <stgao@chromium.org>
Reviewed-by: Chan Li <chanli@chromium.org>
Commit-Queue: Roberto Carrillo <robertocn@chromium.org>

[modify] https://crrev.com/1c48cc219cfdd70c0dfc548e8ce5893eea0097ec/appengine/findit/libs/test/email_util_test.py
[modify] https://crrev.com/1c48cc219cfdd70c0dfc548e8ce5893eea0097ec/appengine/findit/libs/email_util.py

Comment 2 by robert...@chromium.org, Jan 24 2018

Status: Fixed (was: Started)
►

Sign in to add a comment