Changeset 211389 in webkit


Ignore:
Timestamp:
Jan 30, 2017 3:21:57 PM (7 years ago)
Author:
aakash_jain@apple.com
Message:

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.

Location:
trunk/Tools
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/Tools/ChangeLog

    r211388 r211389  
     12017-01-30  Aakash Jain  <aakash_jain@apple.com>
     2
     3        QueueStatusServer should have an explicit timeout for _fetch_url
     4        https://bugs.webkit.org/show_bug.cgi?id=167467
     5
     6        Reviewed by Alexey Proskuryakov.
     7
     8        * Scripts/webkitpy/common/net/statusserver.py:
     9        (StatusServer._fetch_url): Add an explicit timeout of 300s.
     10
    1112017-01-30  Kocsen Chung  <kocsen_chung@apple.com>
    212

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

    r210944 r211389  
    167167        # FIXME: This should use NetworkTransaction's 404 handling instead.
    168168        try:
    169             return urllib2.urlopen(url).read()
     169            return urllib2.urlopen(url, timeout=300).read()
    170170        except urllib2.HTTPError, e:
    171171            if e.code == 404:
Note: See TracChangeset for help on using the changeset viewer.