Fehlerbehebung: Logging-Setup für Multi-Client-Betrieb korrigiert

- Logdateipfad wird vor dem Aufruf von fileConfig() über builtins.log_filename gesetzt
- Entfernt manuelles Patchen des Loghandlers (BaseFilename, Stream etc.) nach dem Laden
- Logging-Konfiguration (logger_client.ini) wird nur noch einmal geladen
- logger_*.ini: Korrektur: interval=midnight ist nicht (mehr) offiziell unterstützt, Änderung auf korrekte Schreibweise (when=midnight, interval=1)
- Ändern der Kommentare auf Englisch (stringent zu anderen Kommentaren)

Author: KoenigMjr

bw_client.py

@@ -30,8 +30,9 @@
 import logging.config
 import argparse
 import os
+import builtins
 
-# Argumente zuerst parsen
+# parsing arguments first - this is needed to load the logging config file with the correct log filename
 parser = argparse.ArgumentParser(prog="bw_client.py",
                                  description="""BOSWatch is a Python Script to receive and
                                  decode german BOS information with rtl_fm and multimon-NG""",
@@ -42,20 +43,13 @@
 parser.add_argument("-t", "--test", help="Start Client with testdata-set", action="store_true")
 args = parser.parse_args()
 
-# Logging-Konfiguration laden
-logging.config.fileConfig(paths.CONFIG_PATH + "logger_client.ini", disable_existing_loggers=False)
-
-# Dynamischer Logdateiname basierend auf YAML-Datei
-yaml_basename = os.path.splitext(args.config)[0]
-log_filename = f"{paths.LOG_PATH}{yaml_basename}.log"
+# set the log filename in the global namespace (mandatory for fileConfig)
+log_filename = f"{paths.LOG_PATH}{os.path.splitext(args.config)[0]}.log"
+builtins.log_filename = log_filename
 
-for handler in logging.getLogger().handlers:
-    if isinstance(handler, logging.handlers.TimedRotatingFileHandler):
-        handler.baseFilename = os.path.abspath(log_filename)
-        handler.stream.close()
-        handler.stream = open(handler.baseFilename, handler.mode)
+logging.config.fileConfig(paths.CONFIG_PATH + "logger_client.ini", disable_existing_loggers=False)
 
-# Placeholder-Logdatei löschen, falls dynamisch umgebogen
+# delete the placeholder log file if it exists and is not the current log file
 placeholder_log = os.path.abspath(os.path.join(paths.LOG_PATH, "client.log"))
 if os.path.abspath(log_filename) != placeholder_log and os.path.isfile(placeholder_log):
     try:

config/logger_client.ini

@@ -30,11 +30,12 @@ keys=file,screen
 
 [handler_file]
 class=handlers.TimedRotatingFileHandler
-interval=midnight
+when=midnight
+interval=1
 backupCount=7
 formatter=complex
 level=ERROR
-args=('log/client.log',)
+args=(log_filename,)
 
 [handler_screen]
 class=StreamHandler

config/logger_server.ini

@@ -30,7 +30,8 @@ keys=file,screen
 
 [handler_file]
 class=handlers.TimedRotatingFileHandler
-interval=midnight
+when=midnight
+interval=1
 backupCount=7
 formatter=complex
 level=ERROR