EWS for security bugs
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 21 Jun 2018 22:14:01 +0000 (22:14 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 21 Jun 2018 22:14:01 +0000 (22:14 +0000)
https://bugs.webkit.org/show_bug.cgi?id=186291
<rdar://problem/40829658>

In r232979, I inadvertently removed a line of code in _parse_authorized_api_keys() to define a local called api_keys.
Add this code back.

* QueueStatusServer/config/authorization.py:
(_parse_authorized_api_keys):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233061 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/QueueStatusServer/config/authorization.py

index e8159bb..a0a2721 100644 (file)
@@ -1,5 +1,17 @@
 2018-06-21  Daniel Bates  <dabates@apple.com>
 
+        EWS for security bugs
+        https://bugs.webkit.org/show_bug.cgi?id=186291
+        <rdar://problem/40829658>
+
+        In r232979, I inadvertently removed a line of code in _parse_authorized_api_keys() to define a local called api_keys.
+        Add this code back.
+
+        * QueueStatusServer/config/authorization.py:
+        (_parse_authorized_api_keys):
+
+2018-06-21  Daniel Bates  <dabates@apple.com>
+
         EWS should not try to post comments or upload result archives to security-sensitive
         bugs unless it has access
         https://bugs.webkit.org/show_bug.cgi?id=186831
index 56d3932..bf9d3f6 100644 (file)
@@ -31,6 +31,7 @@ def _path_to_authorized_api_keys_file():
 
 
 def _parse_authorized_api_keys(file):
+    api_keys = set()
     for line in file:
         line = line.strip()
         if not line or line.startswith("#"):  # Skip empty lines and comments