Provide more intuitive warnings when config file is missing an entry

SACGF · Sep 13, 2023 · c526916 · c526916
1 parent 8a5f15b
commit c526916
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/variantgrid/settings/components/secret_settings.py b/variantgrid/settings/components/secret_settings.py
@@ -127,7 +127,7 @@ def get_secret(key: str, mandatory: bool = True) -> Optional[Any]:
     value, found = _get_nested(parts, _default_settings)
     if found:
         if key != 'DB.port':
-            logging.warning(f'Warning {key} was loaded from default_settings, migrate to settings config or env variable')
+            logging.warning(f"Warning '{key}' not present in config file '{_settings_file()}', using default value '{found}', please migrate")
         return value
 
     if not mandatory:
@@ -136,7 +136,7 @@ def get_secret(key: str, mandatory: bool = True) -> Optional[Any]:
     settings_filename = _settings_file()
     if not os.path.exists(settings_filename):
         raise FileNotFoundError(f"Secret settings file '{settings_filename}' not found")
-    raise ValueError(f"No config value found for {key} in secret settings file '{settings_filename}'")
+    raise ValueError(f"Error '{key}' not present in config file '{settings_filename}'")
 
 
 def get_secrets(prefix: str, leafs: List[str], mandatory: bool = True) -> Dict[str, Any]: