Title: [233061] trunk/Tools
Revision
233061
Author
[email protected]
Date
2018-06-21 15:14:01 -0700 (Thu, 21 Jun 2018)

Log Message

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):

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (233060 => 233061)


--- trunk/Tools/ChangeLog	2018-06-21 21:42:59 UTC (rev 233060)
+++ trunk/Tools/ChangeLog	2018-06-21 22:14:01 UTC (rev 233061)
@@ -1,5 +1,17 @@
 2018-06-21  Daniel Bates  <[email protected]>
 
+        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  <[email protected]>
+
         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

Modified: trunk/Tools/QueueStatusServer/config/authorization.py (233060 => 233061)


--- trunk/Tools/QueueStatusServer/config/authorization.py	2018-06-21 21:42:59 UTC (rev 233060)
+++ trunk/Tools/QueueStatusServer/config/authorization.py	2018-06-21 22:14:01 UTC (rev 233061)
@@ -31,6 +31,7 @@
 
 
 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
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to