[tor-commits] [bridgedb/develop] Add coverage pragma lines to blanket excepts in HTTPServer.

isis at torproject.org isis at torproject.org
Wed Mar 26 05:49:31 UTC 2014


commit 4eb7ef398e2e2c28f19031fca6595c679c613fe9
Author: Isis Lovecruft <isis at torproject.org>
Date:   Wed Mar 26 04:56:31 2014 +0000

    Add coverage pragma lines to blanket excepts in HTTPServer.
    
    They are gross, but we have to have these general excepts everywhere,
    otherwise users could be shown a traceback.
---
 lib/bridgedb/HTTPServer.py |   12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/lib/bridgedb/HTTPServer.py b/lib/bridgedb/HTTPServer.py
index d7b56b3..f4ab592 100644
--- a/lib/bridgedb/HTTPServer.py
+++ b/lib/bridgedb/HTTPServer.py
@@ -64,7 +64,7 @@ _geoipdb = '/usr/share/GeoIP/GeoIP.dat'
 
 try:
     # Make sure we have the database before trying to import the module:
-    if not os.path.isfile(_geoipdb):
+    if not os.path.isfile(_geoipdb):  # pragma: no cover
         raise EnvironmentError("Could not find %r. On Debian-based systems, "\
                                "please install the geoip-database package."
                                % _geoipdb)
@@ -74,7 +74,7 @@ try:
     import pygeoip
     geoip = pygeoip.GeoIP(_geoipdb, flags=pygeoip.MEMORY_CACHE)
     logging.info("GeoIP database loaded")
-except Exception as err:
+except Exception as err:  # pragma: no cover
     logging.warn("Error while loading geoip module: %r" % err)
     geoip = None
 
@@ -162,7 +162,7 @@ class CaptchaProtectedResource(resource.Resource):
         try:
             challenge = request.args['captcha_challenge_field'][0]
             response = request.args['captcha_response_field'][0]
-        except:
+        except Exception:  # pragma: no cover
             return redirectTo(request.URLPath(), request)
         return (challenge, response)
 
@@ -314,7 +314,7 @@ class GimpCaptchaProtectedResource(CaptchaProtectedResource):
             capt.get()
         except captcha.GimpCaptchaError as error:
             logging.error(error)
-        except Exception as error:
+        except Exception as error:  # pragma: no cover
             logging.error("Unhandled error while retrieving Gimp captcha!")
             logging.exception(error)
 
@@ -466,7 +466,7 @@ class ReCaptchaProtectedResource(CaptchaProtectedResource):
                       % (Util.logSafely(clientIP), request.args))
 
         def checkResponse(solution, request):
-            if solution.is_valid:
+            if solution.is_valid:  # pragma: no cover
                 logging.info("Valid CAPTCHA solution from %r."
                              % Util.logSafely(clientIP))
                 return (True, request)
@@ -531,7 +531,7 @@ class WebResourceOptions(resource.Resource):
 
         try:
             rtl = usingRTLLang(request)
-        except Exception as err:
+        except Exception as err:  # pragma: no cover
             logging.exception(err)
             logging.error("The gettext files were not properly installed.")
             logging.info("To install translations, try doing `python " \





More information about the tor-commits mailing list