add new setting to configure Requests timeout

Signed-off-by: tdruez <tdruez@aboutcode.org>

Author: tdruez

dejacode/settings.py

@@ -696,6 +696,9 @@ def get_fake_redis_connection(config, use_strict_redis):
 # during the Django 6.x release cycle.
 URLIZE_ASSUME_HTTPS = env.bool("DEJACODE_URLIZE_ASSUME_HTTPS", default=True)
 
+# Default to 5 seconds.
+DEJACODE_INTEGRATION_REQUESTS_TIMEOUT = env.int("DEJACODE_INTEGRATION_REQUESTS_TIMEOUT", default=5)
+
 if IS_TESTS:
     # Silent the django-axes logging during tests
     LOGGING["loggers"].update({"axes": {"handlers": ["null"]}})

dejacode_toolkit/__init__.py

@@ -21,10 +21,13 @@ def get_settings(var_name, default=None):
     return getenv(var_name) or getattr(settings, var_name, default)
 
 
+REQUESTS_TIMEOUT = get_settings("DEJACODE_INTEGRATION_REQUESTS_TIMEOUT", default=5)
+
+
 def is_service_available(label, session, url, raise_exceptions):
     """Check if a configured integration service is available."""
     try:
-        response = session.head(url, timeout=5)
+        response = session.head(url, timeout=REQUESTS_TIMEOUT)
         response.raise_for_status()
     except requests.exceptions.RequestException as request_exception:
         logger.debug(f"{label} is_available() error: {request_exception}")
@@ -40,7 +43,7 @@ class BaseService:
     settings_prefix = None
     url_field_name = None
     api_key_field_name = None
-    default_timeout = 5
+    default_timeout = REQUESTS_TIMEOUT
 
     def __init__(self, dataspace):
         if not dataspace:

dejacode_toolkit/vulnerablecode.py

@@ -126,7 +126,7 @@ def get_vulnerable_cpes(self, components):
         if not cpes:
             return []
 
-        search_results = self.bulk_search_by_cpes(cpes, timeout=5)
+        search_results = self.bulk_search_by_cpes(cpes)
         if not search_results:
             return []