Fix debug reset - only consider CLI args and env vars, ignore config files to...

Fix debug reset - only consider CLI args and env vars, ignore config files to prevent unwanted debug persistence
parent 145b4cd9
...@@ -33,30 +33,27 @@ def initialize_config(cli_args=None) -> None: ...@@ -33,30 +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: reset to false unless explicitly set via CLI, env var, or config file # Special handling for debug: reset to false unless explicitly set via CLI or env var
# (config files are ignored for debug to prevent persistent debug settings)
debug_explicitly_set = False debug_explicitly_set = False
if cli_args and hasattr(cli_args, 'debug') and cli_args.debug: if cli_args and hasattr(cli_args, 'debug') and cli_args.debug:
debug_explicitly_set = True debug_explicitly_set = True
elif f'VIDAI_DEBUG' in os.environ: if f'VIDAI_DEBUG' in os.environ:
debug_explicitly_set = True
elif 'debug' in initial_config and initial_config['debug'] != DEFAULTS['debug']:
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 via CLI, env, or config file # 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') and cli_args.debug_web: if cli_args and hasattr(cli_args, 'debug_web') and cli_args.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
elif 'debug_web' in initial_config and initial_config['debug_web'] != DEFAULTS['debug_web']:
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 via CLI, env, or config file # 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