rogs/cleanmedia
rogs/cleanmedia
1
0
Fork 0
Code Issues 1 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

More robustness against flawed config files

Browse Source 
1) Don't bail out with an unset CONN_STR if we don't have
   ANY connection_string set in our config file.
2) No need to guard against missing section [media_api], we
   verify its existance a few lines above.
This commit is contained in:
Sebastian Spaeth 2023-12-04 08:55:30 +01:00
parent cf1eb36aa6
commit 3e12ef1b1a
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
cleanmedia
View File

@ -170,9 +170,10 @@ def read_config(conf_file: Union[str, Path]) -> Tuple[Path, str]:
logging.error("Missing section media_api")
exit(1)
CONN_STR = None
if "global" in config and "database" in config["global"]:
CONN_STR = config["global"]["database"].get("connection_string", None)
elif "media_api" in config and "database" in config["media_api"]:
elif "database" in config["media_api"]:
logging.debug("No database section in global, but one in media_api, using that")
CONN_STR = config["media_api"]["database"].get("connection_string", None)