Changeset 55704 in webkit

Timestamp:

Mar 8, 2010 8:21:08 PM (14 years ago)

Author:

ukai@chromium.org

Message:

2010-03-08 Fumitoshi Ukai <​ukai@chromium.org>

Reviewed by Adam Barth.

Fix webkit-patch post and land to work well with security bug
​https://bugs.webkit.org/show_bug.cgi?id=35733

Bugzilla requires authentication to access security bug page,
so call authenticate() if it failed to fetch bug page.

• Scripts/webkitpy/bugzilla.py:

Location:

trunk/WebKitTools

Files:

• ChangeLog (modified) (1 diff)
• Scripts/webkitpy/bugzilla.py (modified) (1 diff)

trunk/WebKitTools/ChangeLog

@@ +1 @@
+2010-03-08  Fumitoshi Ukai  <ukai@chromium.org>
+
+        Reviewed by Adam Barth.
+
+        Fix webkit-patch post and land to work well with security bug
+        https://bugs.webkit.org/show_bug.cgi?id=35733
+
+        Bugzilla requires authentication to access security bug page,
+        so call authenticate() if it failed to fetch bug page.
+
+        * Scripts/webkitpy/bugzilla.py:
+
 2010-03-04  Dirk Pranke  <dpranke@chromium.org>
 

trunk/WebKitTools/Scripts/webkitpy/bugzilla.py

@@ -419 +419 @@
 
     def fetch_bug_dictionary(self, bug_id):
-        return self._parse_bug_page(self._fetch_bug_page(bug_id))
+        try:
+            return self._parse_bug_page(self._fetch_bug_page(bug_id))
+        except:
+            self.authenticate()
+            return self._parse_bug_page(self._fetch_bug_page(bug_id))
 
     # FIXME: A BugzillaCache object should provide all these fetch_ methods.