Merge pull request #2602 from JamieeBond/feat/add-seven-zip-timeout-env-variable

Feat: Add SEVEN_ZIP_TIMEOUT

Author: gantoine

backend/config/__init__.py

@@ -28,6 +28,9 @@ def str_to_bool(value: str) -> bool:
 ASSETS_BASE_PATH: Final = f"{ROMM_BASE_PATH}/assets"
 FRONTEND_RESOURCES_PATH: Final = "/assets/romm/resources"
 
+# SEVEN ZIP
+SEVEN_ZIP_TIMEOUT: Final = int(os.environ.get("SEVEN_ZIP_TIMEOUT", 60))
+
 # DATABASE
 DB_HOST: Final[str | None] = os.environ.get("DB_HOST", "127.0.0.1") or None
 DB_PORT: Final[int | None] = (

backend/utils/archive_7zip.py

@@ -5,7 +5,7 @@
 from collections.abc import Callable
 from pathlib import Path
 
-from config import ROMM_TMP_PATH
+from config import ROMM_TMP_PATH, SEVEN_ZIP_TIMEOUT
 from logger.logger import log
 
 SEVEN_ZIP_PATH = "/usr/bin/7zz"
@@ -30,7 +30,7 @@ def process_file_7z(
             capture_output=True,
             text=True,
             check=True,
-            timeout=60,
+            timeout=SEVEN_ZIP_TIMEOUT,
             shell=False,  # trunk-ignore(bandit/B603): 7z path is hardcoded, args are validated
         )
 
@@ -76,7 +76,7 @@ def process_file_7z(
                 ],
                 capture_output=True,
                 check=True,
-                timeout=60,
+                timeout=SEVEN_ZIP_TIMEOUT,
                 shell=False,  # trunk-ignore(bandit/B603): 7z path is hardcoded, args are validated
             )
 

env.template

@@ -80,6 +80,7 @@ RESCAN_ON_FILESYSTEM_CHANGE_DELAY=5
 # Tasks (optional)
 TASK_TIMEOUT=300
 TASK_RESULT_TTL=86400
+SEVEN_ZIP_TIMEOUT=60
 ENABLE_SCHEDULED_RESCAN=true
 SCHEDULED_RESCAN_CRON=0 3 * * *
 ENABLE_SCHEDULED_UPDATE_SWITCH_TITLEDB=true