From 2b1efe63cf194e7fa207c0576b5ee76971f9f591 Mon Sep 17 00:00:00 2001 From: Chih-Hung Hsieh Date: Thu, 7 Sep 2017 11:55:31 -0700 Subject: [PATCH] Accept alternative email addresses. Some OWNERS files use secondary email addresses, so we only check existence of _account_id and email attributes. Test: run against existing OWNERS files Change-Id: I6f0804a8a90b7a23783c3fe45686d4d973876ed1 --- tools/checkowners.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/checkowners.py b/tools/checkowners.py index b8749550da..1190d30f2c 100755 --- a/tools/checkowners.py +++ b/tools/checkowners.py @@ -34,8 +34,8 @@ def find_address(address): + urllib.quote(address)) echo('Checking email address: ' + address) result = urllib2.urlopen(request).read() - expected = '"email": "' + address + '"' - checked_addresses[address] = (result.find(expected) >= 0) + checked_addresses[address] = ( + result.find('"email":') >= 0 and result.find('"_account_id":') >= 0) return checked_addresses[address]