diff --git a/docs/decisions/0022-spend-limits-constraint.rst b/docs/decisions/0022-spend-limits-constraint.rst deleted file mode 100644 index 75ce5dda..00000000 --- a/docs/decisions/0022-spend-limits-constraint.rst +++ /dev/null @@ -1,3 +0,0 @@ -0022 Constraint on `spend-limit` based on total deposits -******************************************** - diff --git a/enterprise_access/apps/api_client/constants.py b/enterprise_access/apps/api_client/constants.py new file mode 100644 index 00000000..05901586 --- /dev/null +++ b/enterprise_access/apps/api_client/constants.py @@ -0,0 +1,16 @@ +""" +Constants for API client +""" +import backoff +from requests.exceptions import ConnectionError as RequestsConnectionError +from requests.exceptions import HTTPError +from requests.exceptions import Timeout as RequestsTimeoutError + +# wait_gen references the field in the backoff decorator representing +# the backoff strategy for retries +wait_gen = backoff.expo + +exception = ( + RequestsConnectionError, + RequestsTimeoutError, +) diff --git a/enterprise_access/apps/api_client/enterprise_catalog_client.py b/enterprise_access/apps/api_client/enterprise_catalog_client.py index 5a73714a..f1b1eb9a 100644 --- a/enterprise_access/apps/api_client/enterprise_catalog_client.py +++ b/enterprise_access/apps/api_client/enterprise_catalog_client.py @@ -1,10 +1,11 @@ """ API client for enterprise-catalog service. """ - +import backoff from django.conf import settings from enterprise_access.apps.api_client.base_oauth import BaseOAuthClient +from enterprise_access.apps.api_client.constants import exception, wait_gen class EnterpriseCatalogApiClient(BaseOAuthClient): @@ -14,6 +15,7 @@ class EnterpriseCatalogApiClient(BaseOAuthClient): api_base_url = settings.ENTERPRISE_CATALOG_URL + '/api/v1/' enterprise_catalog_endpoint = api_base_url + 'enterprise-catalogs/' + @backoff.on_exception(wait_gen, exception) def contains_content_items(self, catalog_uuid, content_ids): """ Check whether the specified enterprise catalog contains the given content. @@ -33,6 +35,7 @@ def contains_content_items(self, catalog_uuid, content_ids): response_json = response.json() return response_json.get('contains_content_items', False) + @backoff.on_exception(wait_gen, exception) def catalog_content_metadata(self, catalog_uuid, content_keys, traverse_pagination=True, **kwargs): """ Returns a list of requested content metadata records for the given catalog_uuid. @@ -68,6 +71,7 @@ def catalog_content_metadata(self, catalog_uuid, content_keys, traverse_paginati response.raise_for_status() return response.json() + @backoff.on_exception(wait_gen, exception) def get_content_metadata_count(self, catalog_uuid): """ Returns the count of content metadata for a catalog.