Merge branch 'feature/retry-fix' into 'develop'

Feature/retry fix

See merge request core/sevenbridges-python!114

Author: Borislav Dugosija

sevenbridges/http/error_handlers.py

@@ -43,6 +43,9 @@ def maintenance_sleeper(api, response, sleep=300):
     :param sleep: Time to sleep in between the requests.
     """
     while response.status_code == 503:
+        content_type = response.headers.get('Content-Type')
+        if content_type is None or 'application/json' not in content_type:
+            return response
         logger.info(
             'Service unavailable: Response=[%s]',
             response.__dict__

tests/test_error_handlers.py

@@ -37,6 +37,7 @@ def test_rate_limit_sleeper(api):
 def test_maintenance_sleeper_invalid_json(api):
     resp503 = requests.Response()
     resp503.status_code = 503
+    resp503.headers = {'Content-Type': 'application/json'}
 
     api._session = MockSession([resp503])
     with pytest.raises(JSONDecodeError):
@@ -46,6 +47,7 @@ def test_maintenance_sleeper_invalid_json(api):
 def test_maintenance_sleeper(api):
     resp503 = requests.Response()
     resp503.status_code = 503
+    resp503.headers = {'Content-Type': 'application/json'}
     resp503._content = b'{"code": 0}'
     resp200 = requests.Response()
     resp200.status_code = 200