Better error handling.

kaboramb · Aug 11, 2015 · 2192344 · 2192344
1 parent e9f1ea9
commit 2192344
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 3 deletions.
diff --git a/modules/sfp_coderepo.py b/modules/sfp_coderepo.py
@@ -94,7 +94,11 @@ def handleEvent(self, event):
             self.sf.error("Unable to fetch " + url, False)
             failed = True
 
-        ret = json.loads(res['content'])
+        try:
+            ret = json.loads(res['content'])
+        except BaseException as e:
+            ret = None
+
         if ret == None:
             self.sf.error("Unable to process empty response from Github for: " + \
                           name, False)

diff --git a/modules/sfp_duckduckgo.py b/modules/sfp_duckduckgo.py
@@ -70,7 +70,11 @@ def handleEvent(self, event):
             self.sf.error("Unable to fetch " + url, False)
             return None
 
-        ret = json.loads(res['content'])
+        try:
+            ret = json.loads(res['content'])
+        except BaseException as e:
+            ret = None
+
         if ret == None:
             self.sf.error("Unable to process empty response from DuckDuckGo for: " + \
                           eventData, False)

diff --git a/modules/sfp_historic.py b/modules/sfp_historic.py
@@ -122,7 +122,11 @@ def handleEvent(self, event):
                 self.sf.error("Unable to fetch " + url, False)
                 continue
 
-            ret = json.loads(res['content'])
+            try:
+                ret = json.loads(res['content'])
+            except BaseException as e:
+                ret = None
+
             if ret == None:
                 self.sf.error("Unable to process empty response from archive.org: " + \
                               eventData, False)