From 564ac757d47cd95ff412d76fa4944646a8f1beab Mon Sep 17 00:00:00 2001 From: Jens Gohl Date: Mon, 3 Feb 2025 11:46:12 +0100 Subject: [PATCH] Increase timeout for scripts that are run with login to 50 seconds, the default of 5 seconds proven to short. --- CHANGELOG.md | 6 +++++- app/core/core.py | 3 ++- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index c4a4113..9ff2148 100644 --- 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 index e7fe82f..f6870a5 100644 --- 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