manager/logging.py: added constants for logging targets

diff --git a/python/knot_resolver/manager/logging.py b/python/knot_resolver/manager/logging.py
index 087c392dca99031b9bdf58e100ac33b39880a6b8..054dd0754513a2e2324f441d05dc0b6721b78582 100644 (file)
--- a/python/knot_resolver/manager/logging.py
+++ b/python/knot_resolver/manager/logging.py
@@ -10,6 +10,10 @@ from knot_resolver.manager.config_store import ConfigStore, only_on_real_changes
 
 from .constants import LOGGING_LEVEL_STARTUP
 
+STDOUT = "stdout"
+SYSLOG = "syslog"
+STDERR = "stderr"
+
 NOTICE_LEVEL = (logging.WARNING + logging.INFO) // 2
 NOTICE_NAME = "NOTICE"
 
@@ -57,10 +61,10 @@ def get_log_format(config: KresConfig) -> str:
         return "[%(levelname)s] %(name)s: %(message)s"
     # In this case, we are running standalone during inicialization and we need to add a prefix to each line
     # by ourselves to make it consistent
-    assert config.logging.target != "syslog"
+    assert config.logging.target != SYSLOG
     stream = ""
-    if config.logging.target == "stderr":
-        stream = " (stderr)"
+    if config.logging.target == STDERR:
+        stream = f" ({STDERR})"
 
     pid = os.getpid()
     return f"%(asctime)s manager[{pid}]{stream}: [%(levelname)s] %(name)s: %(message)s"
@@ -86,13 +90,13 @@ async def _set_logging_handler(config: KresConfig) -> None:
         target = "stdout"
 
     handler: logging.Handler
-    if target == "syslog":
+    if target == SYSLOG:
         handler = logging.handlers.SysLogHandler(address="/dev/log")
         handler.setFormatter(logging.Formatter("%(name)s: %(message)s"))
-    elif target == "stdout":
+    elif target == STDOUT:
         handler = logging.StreamHandler(sys.stdout)
         handler.setFormatter(logging.Formatter(get_log_format(config)))
-    elif target == "stderr":
+    elif target == STDERR:
         handler = logging.StreamHandler(sys.stderr)
         handler.setFormatter(logging.Formatter(get_log_format(config)))
     else: