From 6fb21c62989200dde12a56456d5cd698e6d78d19 Mon Sep 17 00:00:00 2001 From: Sam Skillman Date: Mon, 28 Aug 2017 09:28:08 -0600 Subject: [PATCH 1/2] Allow for additional response codes, and retry 429, 504. --- descarteslabs/services/service.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/descarteslabs/services/service.py b/descarteslabs/services/service.py index 9ec2f2ed..44f8861f 100644 --- a/descarteslabs/services/service.py +++ b/descarteslabs/services/service.py @@ -35,7 +35,7 @@ def request(self, method, url, **kwargs): resp = super(WrappedSession, self).request(method, self.base_url + url, **kwargs) - if resp.status_code == 200: + if resp.status_code >= 200 and resp.status_code < 400: return resp elif resp.status_code == 400: raise BadRequestError(resp.text) @@ -61,7 +61,7 @@ class Service: 'HEAD', 'TRACE', 'GET', 'POST', 'PUT', 'OPTIONS', 'DELETE' ]), - status_forcelist=[500, 502, 503]) + status_forcelist=[429, 500, 502, 503, 504]) ADAPTER = HTTPAdapter(max_retries=RETRY_CONFIG) From 593fcb22d13606013bd33e75635b7099b6395ce4 Mon Sep 17 00:00:00 2001 From: Sam Skillman Date: Mon, 28 Aug 2017 09:31:51 -0600 Subject: [PATCH 2/2] No retry on 429 --- descarteslabs/services/service.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/descarteslabs/services/service.py b/descarteslabs/services/service.py index 44f8861f..01d8a1a2 100644 --- a/descarteslabs/services/service.py +++ b/descarteslabs/services/service.py @@ -61,7 +61,7 @@ class Service: 'HEAD', 'TRACE', 'GET', 'POST', 'PUT', 'OPTIONS', 'DELETE' ]), - status_forcelist=[429, 500, 502, 503, 504]) + status_forcelist=[500, 502, 503, 504]) ADAPTER = HTTPAdapter(max_retries=RETRY_CONFIG)