Fix debug reset logic - only consider CLI args and env vars as explicit,...

Fix debug reset logic - only consider CLI args and env vars as explicit, ignore config file settings
parent 5b39ec90
...@@ -33,31 +33,27 @@ def initialize_config(cli_args=None) -> None: ...@@ -33,31 +33,27 @@ def initialize_config(cli_args=None) -> None:
initial_config = load_initial_config(cli_args) initial_config = load_initial_config(cli_args)
# Special handling for debug: if not explicitly set via CLI, config file, or env var, # Special handling for debug: reset to false unless explicitly set via CLI or env var
# reset it to false to prevent database-persisted debug settings from staying enabled # (ignore config file settings to prevent persistent debug from config files)
debug_explicitly_set = False debug_explicitly_set = False
if cli_args and hasattr(cli_args, 'debug'): if cli_args and hasattr(cli_args, 'debug') and cli_args.debug:
debug_explicitly_set = True debug_explicitly_set = True
elif 'debug' in initial_config and initial_config['debug'] != DEFAULTS['debug']: if f'VIDAI_DEBUG' in os.environ:
debug_explicitly_set = True
elif f'VIDAI_DEBUG' in os.environ:
debug_explicitly_set = True debug_explicitly_set = True
if not debug_explicitly_set: if not debug_explicitly_set:
# Reset debug to false if not explicitly set # Reset debug to false if not explicitly set via CLI or env
initial_config['debug'] = 'false' initial_config['debug'] = 'false'
# Special handling for debug_web: same logic as debug # Special handling for debug_web: same logic as debug
debug_web_explicitly_set = False debug_web_explicitly_set = False
if cli_args and hasattr(cli_args, 'debug_web'): if cli_args and hasattr(cli_args, 'debug_web') and cli_args.debug_web:
debug_web_explicitly_set = True
elif 'debug_web' in initial_config and initial_config['debug_web'] != DEFAULTS['debug_web']:
debug_web_explicitly_set = True debug_web_explicitly_set = True
elif f'VIDAI_DEBUG_WEB' in os.environ: if f'VIDAI_DEBUG_WEB' in os.environ:
debug_web_explicitly_set = True debug_web_explicitly_set = True
if not debug_web_explicitly_set: if not debug_web_explicitly_set:
# Reset debug_web to false if not explicitly set # Reset debug_web to false if not explicitly set via CLI or env
initial_config['debug_web'] = 'false' initial_config['debug_web'] = 'false'
# Store all environment variables that start with VIDAI_ # Store all environment variables that start with VIDAI_
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment