From 315992c37a6577da5fde92ae64aa7aaa96e3625c Mon Sep 17 00:00:00 2001 From: sHa Date: Fri, 20 Mar 2026 14:35:45 +0200 Subject: [PATCH] Refactor environment variable validation in config.py for improved error messaging; update Docker Compose restart policy to on-failure --- app/config.py | 9 +++++---- compose.yml | 2 +- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/app/config.py b/app/config.py index 43b1de9..4647a46 100644 --- a/app/config.py +++ b/app/config.py @@ -5,7 +5,8 @@ REDMINE_API_KEY = os.environ.get("REDMINE_API_KEY", "") def validate(): - if not REDMINE_URL or not REDMINE_API_KEY: - raise RuntimeError( - "REDMINE_URL and REDMINE_API_KEY environment variables must be set." - ) + missing = [name for name, val in [("REDMINE_URL", REDMINE_URL), ("REDMINE_API_KEY", REDMINE_API_KEY)] if not val] + if missing: + print(f"ERROR: Required environment variable(s) not set: {', '.join(missing)}") + print("Set them in your .env file or pass via -e flags. Exiting.") + raise SystemExit(0) diff --git a/compose.yml b/compose.yml index 3ef922e..d2d7f0d 100644 --- a/compose.yml +++ b/compose.yml @@ -9,4 +9,4 @@ services: MCP_TRANSPORT: http MCP_HOST: 0.0.0.0 MCP_PORT: 8000 - restart: unless-stopped + restart: on-failure