From 94e7b2b4d211e8f55866c8a93af3fb1e2916b53b Mon Sep 17 00:00:00 2001 From: Shroominic Date: Thu, 13 Nov 2025 16:03:56 +0800 Subject: [PATCH] Fix settings override bug: allow False and 0 from database Fixes #217 - Database values of False and 0 are now properly respected instead of being ignored. Removed 'and v' check that incorrectly treated these legitimate config values as 'empty'. Now only truly empty values (None, empty string, empty list, empty dict) are ignored in favor of env. --- routstr/core/settings.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/routstr/core/settings.py b/routstr/core/settings.py index eb8e59a9..cd59758c 100644 --- a/routstr/core/settings.py +++ b/routstr/core/settings.py @@ -235,7 +235,7 @@ class SettingsService: merged_dict: dict[str, Any] = dict(env_resolved.dict()) merged_dict.update( - {k: v for k, v in db_json.items() if v not in (None, "", []) and v} + {k: v for k, v in db_json.items() if v not in (None, "", [], {})} ) # Ensure primary_mint is consistent with cashu_mints if not explicitly set