Do not create an Arkindex client on import

Closes #6 (closed)

Requires arkindex/api-client!234 (merged)

teklia_toolbox/requests.py

@@ -4,18 +4,14 @@ from urllib.parse import urlparse
 
 import requests
 import urllib3
-from apistar.exceptions import ErrorResponse
 from tenacity import (
-    before_sleep_log,
     retry,
-    retry_if_exception,
     retry_if_exception_type,
     stop_after_attempt,
-    wait_exponential,
     wait_fixed,
 )
 
-from arkindex import ArkindexClient, options_from_env
+from arkindex import ArkindexClient
 
 logger = logging.getLogger(__name__)
 
@@ -31,9 +27,7 @@ def should_verify_cert(url: str) -> bool:
     return not host.endswith("ark.localhost")
 
 
-def _get_arkindex_client() -> ArkindexClient:
-    options = options_from_env()
-
+def get_arkindex_client(**options) -> ArkindexClient:
     # Skip SSL verification in Arkindex API client for local development hosts
     verify = should_verify_cert(options.get("base_url"))
     if not verify:
@@ -43,40 +37,6 @@ def _get_arkindex_client() -> ArkindexClient:
     return ArkindexClient(verify=verify, **options)
 
 
-DEFAULT_CLIENT = _get_arkindex_client()
-
-
-def _is_500_error(exc: Exception) -> bool:
-    """
-    Check if an Arkindex API error is a 50x
-    This is used to retry most API calls implemented here
-    """
-    if not isinstance(exc, ErrorResponse):
-        return False
-
-    return 500 <= exc.status_code < 600
-
-
-@retry(
-    retry=retry_if_exception(_is_500_error),
-    wait=wait_exponential(multiplier=2, min=3),
-    reraise=True,
-    stop=stop_after_attempt(5),
-    before_sleep=before_sleep_log(logger, logging.INFO),
-)
-def retried_request(*args, **kwargs):
-    """
-    Proxy all Arkindex API requests with a retry mechanism
-    in case of 50X errors
-    The same API call will be retried 5 times, with an exponential sleep time
-    going through 3, 4, 8 and 16 seconds of wait between call.
-    If the 5th call still gives a 50x, the exception is re-raised
-    and the caller should catch it
-    Log messages are displayed before sleeping (when at least one exception occurred)
-    """
-    return DEFAULT_CLIENT.request(*args, **kwargs)
-
-
 # Time to wait before retrying the IIIF image information fetching
 HTTP_GET_RETRY_BACKOFF = 10