Timeout added for opener

Fixes #20

Author: prabhakar267

python_http_client/client.py

@@ -143,18 +143,20 @@ def _build_client(self, name=None):
                       url_path=url_path,
                       append_slash=self.append_slash)
 
-    def _make_request(self, opener, request):
+    def _make_request(self, opener, request, timeout=4):
         """Make the API call and return the response. This is separated into
            it's own function, so we can mock it easily for testing.
 
         :param opener:
         :type opener:
         :param request: url payload to request
         :type request: urllib.Request object
+        :param timeout: timeout for request in seconds, default 4 sec
+        :type integer:
         :return: urllib response
         """
         try:
-            return opener.open(request)
+            return opener.open(request, timeout=timeout)
         except HTTPError as err:
             exc = handle_error(err)
             exc.__cause__ = None
@@ -224,7 +226,7 @@ def http_request(*_, **kwargs):
                 if data and not ('Content-Type' in self.request_headers):
                     request.add_header('Content-Type', 'application/json')
                 request.get_method = lambda: method
-                return Response(self._make_request(opener, request))
+                return Response(self._make_request(opener, request, timeout=4))
             return http_request
         else:
             # Add a segment to the URL

tests/test_unit.py

@@ -59,7 +59,7 @@ def __init__(self, host, response_code):
         self.response_code = 200
         Client.__init__(self, host)
 
-    def _make_request(self, opener, request):
+    def _make_request(self, opener, request, timeout):
         if 200 <= self.response_code <299:   # if successsful code
             return MockResponse(self.response_code)
         else: