Congegni/sintonia
7
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fixed listener stats thread hanging on network errors

Browse source 
* Same fix as before, urlopen() wasn't being used with a timeout.
* Fixes CC-5472 and SAAS-394 aka. listener stats breaking all the time
This commit is contained in:
Albert Santoni 2014-02-05 17:29:15 -05:00
parent 296adfdb24
commit b4e7b9324b
1 changed files with 3 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

4
python_apps/pypo/listenerstat.py
View file

@ -10,6 +10,8 @@ import time
from api_clients import api_client from api_clients import api_client
class ListenerStat(Thread): class ListenerStat(Thread):
HTTP_REQUEST_TIMEOUT = 30 # 30 second HTTP request timeout
def __init__(self, logger=None): def __init__(self, logger=None):
Thread.__init__(self) Thread.__init__(self)
self.api_client = api_client.AirtimeApiClient() self.api_client = api_client.AirtimeApiClient()
@ -46,7 +48,7 @@ class ListenerStat(Thread):
url=url, url=url,
headers=header) headers=header)
f = urllib2.urlopen(req) f = urllib2.urlopen(req, timeout=ListenerStat.HTTP_REQUEST_TIMEOUT)
document = f.read() document = f.read()
return document return document