added console logging

.env.example

@@ -18,6 +18,7 @@ LOG_FILE=/var/lib/rrd/netatmo_service.log
 LOG_LEVEL=INFO
 LOG_MAX_BYTES=5242880
 LOG_BACKUP_COUNT=5
+LOG_CONSOLE=true
 POLL_INTERVAL=600
 START_COLLECTOR=true
 

README.md

@@ -79,7 +79,9 @@ open <http://localhost:5000/w/> instead.
 failure, HTTP status, endpoint, and duration without credentials. Collector,
 Flask, uncaught main-thread, and uncaught worker-thread exceptions are written to
 the same log. `LOG_MAX_BYTES` defaults to 5 MiB and `LOG_BACKUP_COUNT` to five.
-The service account must have write permission on the log directory.
+The service account must have write permission on the log directory. Console
+logging remains enabled by default so `python wsgi.py` displays its listening
+address; set `LOG_CONSOLE=false` to use only the logfile.
 
 ## HTTP API
 

netatmo_service/app.py

@@ -35,15 +35,16 @@ class PrefixMiddleware:
 def configure_logging(app: Flask) -> None:
     log_path = app.config["LOG_FILE"]
     log_path.parent.mkdir(parents=True, exist_ok=True)
+    formatter = logging.Formatter(
+        "%(asctime)s %(levelname)s %(name)s [%(threadName)s] %(message)s"
+    )
     handler = RotatingFileHandler(
         log_path,
         maxBytes=app.config["LOG_MAX_BYTES"],
         backupCount=app.config["LOG_BACKUP_COUNT"],
         encoding="utf-8",
     )
-    handler.setFormatter(logging.Formatter(
+    handler.setFormatter(formatter)
-        "%(asctime)s %(levelname)s %(name)s [%(threadName)s] %(message)s"
-    ))
     root = logging.getLogger()
     root.setLevel(app.config["LOG_LEVEL"])
     # Avoid duplicate handlers when an app factory is called repeatedly in tests.
@@ -57,6 +58,15 @@ def configure_logging(app: Flask) -> None:
     else:
         handler.close()
 
+    if app.config["LOG_CONSOLE"] and not any(
+        isinstance(item, logging.StreamHandler)
+        and not isinstance(item, logging.FileHandler)
+        for item in root.handlers
+    ):
+        console = logging.StreamHandler()
+        console.setFormatter(formatter)
+        root.addHandler(console)
+
     def uncaught_exception(exception_type, exception, traceback):
         if issubclass(exception_type, KeyboardInterrupt):
             return sys.__excepthook__(exception_type, exception, traceback)

netatmo_service/config.py

@@ -25,6 +25,7 @@ class Config:
     LOG_LEVEL = os.getenv("LOG_LEVEL", "INFO").upper()
     LOG_MAX_BYTES = int(os.getenv("LOG_MAX_BYTES", str(5 * 1024 * 1024)))
     LOG_BACKUP_COUNT = int(os.getenv("LOG_BACKUP_COUNT", "5"))
+    LOG_CONSOLE = _bool("LOG_CONSOLE", True)
     NETATMO_TOKEN_FILE = Path(
         os.getenv("NETATMO_TOKEN_FILE", str(RRD_FOLDER / "netatmo_tokens.json"))
     ).expanduser().resolve()