Malin/krawl.es
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #51 from BlessedRebuS/fix/env-config-override

Browse Source 
Add logging for environment variable overrides in config
This commit is contained in:
Longhi Matteo
2026-01-23 22:29:04 +01:00
committed by GitHub
parent 6ffe2ac2e4 4addf41a5b
commit 02e2aee950
1 changed files with 22 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

34
src/config.py
View File

@@ -7,6 +7,7 @@ from pathlib import Path
from typing import Optional, Tuple from typing import Optional, Tuple
from zoneinfo import ZoneInfo from zoneinfo import ZoneInfo
import time import time
from logger import get_app_logger
import yaml import yaml
@@ -157,18 +158,27 @@ def override_config_from_env(config: Config = None):
env_var = __get_env_from_config(field) env_var = __get_env_from_config(field)
if env_var in os.environ: if env_var in os.environ:
field_type = config.__dataclass_fields__[field].type
env_value = os.environ[env_var] get_app_logger().info(
if field_type == int: f"Overriding config '{field}' from environment variable '{env_var}'"
setattr(config, field, int(env_value)) )
elif field_type == float: try:
setattr(config, field, float(env_value)) field_type = config.__dataclass_fields__[field].type
elif field_type == Tuple[int, int]: env_value = os.environ[env_var]
parts = env_value.split(",") if field_type == int:
if len(parts) == 2: setattr(config, field, int(env_value))
setattr(config, field, (int(parts[0]), int(parts[1]))) elif field_type == float:
else: setattr(config, field, float(env_value))
setattr(config, field, env_value) elif field_type == Tuple[int, int]:
parts = env_value.split(",")
if len(parts) == 2:
setattr(config, field, (int(parts[0]), int(parts[1])))
else:
setattr(config, field, env_value)
except Exception as e:
get_app_logger().error(
f"Error overriding config '{field}' from environment variable '{env_var}': {e}"
)
_config_instance = None _config_instance = None