Title: [288322] trunk/Tools
Revision
288322
Author
[email protected]
Date
2022-01-20 13:36:40 -0800 (Thu, 20 Jan 2022)

Log Message

[ews] Load credentials from passwords.json in master.cfg (Follow-up)
https://bugs.webkit.org/show_bug.cgi?id=235296

Reviewed by Aakash Jain.

* CISupport/ews-build/master.cfg:
(load_password): Define default value.

Canonical link: https://commits.webkit.org/246234@main

Modified Paths

Diff

Modified: trunk/Tools/CISupport/ews-build/master.cfg (288321 => 288322)


--- trunk/Tools/CISupport/ews-build/master.cfg	2022-01-20 21:26:46 UTC (rev 288321)
+++ trunk/Tools/CISupport/ews-build/master.cfg	2022-01-20 21:36:40 UTC (rev 288322)
@@ -10,15 +10,15 @@
 from events import Events, GitHubEventHandlerNoEdits
 
 
-def load_password(name):
+def load_password(name, default=None):
     if os.getenv(name):
         return os.getenv(name)
     try:
         passwords = json.load(open('passwords.json'))
-        return passwords.get(name, None)
+        return passwords.get(name, default)
     except Exception as e:
         print('Error in finding {} in passwords.json'.format(name))
-        return None
+        return default
 
 
 # This is work-around for Twisted having a small size limit for patches. See https://bugs.webkit.org/show_bug.cgi?id=198851#c5

Modified: trunk/Tools/ChangeLog (288321 => 288322)


--- trunk/Tools/ChangeLog	2022-01-20 21:26:46 UTC (rev 288321)
+++ trunk/Tools/ChangeLog	2022-01-20 21:36:40 UTC (rev 288322)
@@ -1,3 +1,13 @@
+2022-01-20  Jonathan Bedard  <[email protected]>
+
+        [ews] Load credentials from passwords.json in master.cfg (Follow-up)
+        https://bugs.webkit.org/show_bug.cgi?id=235296
+
+        Reviewed by Aakash Jain.
+
+        * CISupport/ews-build/master.cfg:
+        (load_password): Define default value.
+
 2022-01-20  Elliott Williams  <[email protected]>
 
         [XCBuild] Build via the workspace with USE_WORKSPACE=YES
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to