Introduce fetch translations timeout setting

Author: foteinigk

README.md

@@ -43,6 +43,40 @@ To learn more about using Transifex Python toolkit check:
 * For a general overview visit [Transifex Native overview](https://developers.transifex.com/docs/native?utm_campaign=tx-native&utm_source=github&utm_medium=link)
 * For some common questions & answers check our [Transifex Native community](https://community.transifex.com/c/transifex-native/17)
 
+# Django settings reference
+
+The Transifex Native Django sdk is controlled via a set of configuration options defined in Django settings:
+
+- TRANSIFEX_TOKEN: API token that connects your application to a Transifex project.
+Must be set for both pushing source strings and fetching translations.
+
+- TRANSIFEX_SECRET: Secret used together with the token for authenticated operations against CDS (e.g. pushing source content, invalidating cache).
+
+- TRANSIFEX_CDS_HOST: Override the default CDS host (https://cds.svc.transifex.net).
+
+- TRANSIFEX_FILTER_STATUS: Optional CDS filter[status] parameter used when fetching translations (e.g. "reviewed", "proofread"). If not set, CDS returns all available statuses.
+
+- TRANSIFEX_FILTER_TAGS: Optional CDS filter[tags] parameter used when fetching translations. Use this to limit fetched content to specific tags.
+
+- TRANSIFEX_MISSING_POLICY: Custom “missing translation” policy class.
+Defaults to the built-in SourceStringPolicy when not provided.
+
+- TRANSIFEX_ERROR_POLICY: Custom error handling policy class.
+Defaults to SourceErrorPolicy.
+
+- TRANSIFEX_CACHE
+Custom cache implementation. Defaults to an in-memory cache (MemoryCache) if not provided. Can be used to integrate with a shared cache (e.g. Redis, memcached).
+
+- SKIP_TRANSLATIONS_SYNC: If True, disables automatic translation sync (OTA) for this environment.
+
+- TRANSIFEX_SYNC_INTERVAL: Interval in seconds for the background sync daemon that fetches translations. Default: 30 * 60 (30 minutes). Set to 0 to disable periodic sync and only fetch on startup.
+
+- TRANSIFEX_FETCH_ALL_LANGUAGES: When True, fetch translations for all languages configured in CDS. When False (default), only fetch translations for languages listed in Django’s LANGUAGES setting.
+
+- TRANSIFEX_FETCH_TEMEOUT:
+Maximum time in seconds to wait when fetching translations or locales from CDS.
+0 (default) = no global timeout.
+
 # License
 
 Licensed under Apache License 2.0, see `LICENSE` file.

transifex/native/cds.py

@@ -2,6 +2,7 @@
 import sys
 import time
 from urllib.parse import urlencode
+from transifex.native.django.settings import TRANSIFEX_FETCH_TEMEOUT
 
 import requests
 from transifex.native.consts import (KEY_CHARACTER_LIMIT,
@@ -364,16 +365,33 @@ def _get_headers(self, use_secret=False, etag=None):
 
     def retry_get_request(self, *args, **kwargs):
         """ Resilient function for GET requests """
-        retries, last_response_status = 0, 202
-        while (last_response_status == 202 or
-                500 <= last_response_status < 600 and
-                retries < MAX_RETRIES):
-
-            if 500 <= last_response_status < 600:
-                retries += 1
-                time.sleep(retries * RETRY_DELAY_SEC)
+        retries_5xx = 0
+        last_response_status = 202
+        start_ts = time.time()
+        max_total_seconds = TRANSIFEX_FETCH_TEMEOUT
 
+        while True:
             response = requests.get(*args, **kwargs)
             last_response_status = response.status_code
 
-        return response
+            # Success or non-retryable status -> return immediately
+            if last_response_status < 500 and last_response_status != 202:
+                return response
+
+            # 202 handling
+            if last_response_status == 202:
+                time.sleep(RETRY_DELAY_SEC)
+
+            # 5xx handling
+            elif 500 <= last_response_status < 600:
+                retries_5xx += 1
+                if retries_5xx > MAX_RETRIES:
+                    return response
+                time.sleep(retries_5xx * RETRY_DELAY_SEC)
+
+            # Timeout handling
+            if (
+                max_total_seconds > 0 and
+                (time.time() - start_ts) >= max_total_seconds
+            ):
+                return response

transifex/native/django/settings.py

@@ -16,3 +16,7 @@
 TRANSIFEX_FETCH_ALL_LANGUAGES = getattr(settings,
                                         'TRANSIFEX_FETCH_ALL_LANGUAGES',
                                         False)
+TRANSIFEX_FETCH_TEMEOUT = getattr(settings,
+                                  'TRANSIFEX_FETCH_TEMEOUT',
+                                  0)
+