QueueStatusServer should have an explicit timeout for _fetch_url

https://bugs.webkit.org/show_bug.cgi?id=167467

Reviewed by Alexey Proskuryakov.

* Scripts/webkitpy/common/net/statusserver.py:
(StatusServer._fetch_url): Add an explicit timeout of 300s.


Canonical link: https://commits.webkit.org/184630@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@211389 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog

@@ -1,3 +1,13 @@
+2017-01-30  Aakash Jain  <aakash_jain@apple.com>
+
+        QueueStatusServer should have an explicit timeout for _fetch_url
+        https://bugs.webkit.org/show_bug.cgi?id=167467
+
+        Reviewed by Alexey Proskuryakov.
+
+        * Scripts/webkitpy/common/net/statusserver.py:
+        (StatusServer._fetch_url): Add an explicit timeout of 300s.
+
 2017-01-30  Kocsen Chung  <kocsen_chung@apple.com>
 
         Add support for Trac instances that host multiple projects.

Tools/Scripts/webkitpy/common/net/statusserver.py

@@ -166,7 +166,7 @@ def update_svn_revision(self, svn_revision_number, broken_bot):
     def _fetch_url(self, url):
         # FIXME: This should use NetworkTransaction's 404 handling instead.
         try:
-            return urllib2.urlopen(url).read()
+            return urllib2.urlopen(url, timeout=300).read()
         except urllib2.HTTPError, e:
             if e.code == 404:
                 return None