Handle bad HTTP responses properly

urlopen raises a HTTPError upon failure, which didn't go through
handleresponseerror().

The original handling is left in case there are obscure conditions
where urlopen doesn't raise a HTTPError.

Author: janverb

pyPostcode/__init__.py

@@ -9,8 +9,9 @@
 
 try:
     from urllib.request import urlopen, Request  # for Python 3
+    from urllib.error import HTTPError
 except ImportError:
-    from urllib2 import urlopen, Request  # for Python 2
+    from urllib2 import urlopen, Request, HTTPError  # for Python 2
 
 import json
 import logging
@@ -63,9 +64,12 @@ def request(self, path=None):
             "X-Api-Key": self.api_key,
         }
 
-        result = urlopen(Request(
-            self.url + path, headers=headers,
-        ))
+        try:
+            result = urlopen(Request(
+                self.url + path, headers=headers,
+            ))
+        except HTTPError as err:
+            self.handleresponseerror(err.code)
 
         if result.getcode() != 200:
             self.handleresponseerror(result.getcode())