Increase timeout for scripts that are run with login to 50 seconds, t…

…he default of 5 seconds proven to short.
otto-de · Feb 3, 2025 · 564ac75 · 564ac75
1 parent f840d7b
commit 564ac75
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -19,6 +19,10 @@ Release dates will be in YYYY-MM-DD format.
 
 ## Unreleased
 
+### Fixed
+- Increase timeout for scripts that are run with login to 50 seconds, the default of 5 seconds proven to short.
+
+
 ## 8.1.0 - 2025-01-23
 
 ### Added
@@ -30,7 +34,7 @@ Release dates will be in YYYY-MM-DD format.
 - Fixed setting of access-keys.
 - Fixed display of profile group icons.
 
-## 8.0.2 - 2025-01-20
+## 8.0.2 - 2025-01-20 (yanked)
 
 ### Fixed
 - Fix an issue where mfa dialog was opened from another thread and caused a crash.

diff --git a/app/core/core.py b/app/core/core.py
@@ -218,8 +218,9 @@ def run_script(profile_group: ProfileGroup) -> Result:
             result.error(f'{profile_group.script} not found')
             return result
 
-        shell_output = shell.run(profile_group.script)
+        shell_output = shell.run(command=profile_group.script, timeout=60)
         if not shell_output:
+            logger.error(f'script output:\n{shell_output}')
             result.error('script failed')
             return result