Merge pull request #34 from FireTail-io/hotfix-handler-duplicates

Hotfix adding duplicating handlers registered to loggers

firetail/logger.py

@@ -1,20 +1,24 @@
 import logging
 import sys
+from threading import Lock
 
-
-def get_logger(debug):
-    return __get_logger(debug, __name__)
+configured_loggers_lock = Lock()
+configured_loggers = set()
 
 
 def get_stdout_logger(debug):
-    stdout_logger = __get_logger(debug, __name__ + "_stdout", logging.StreamHandler(sys.stdout))
-    stdout_logger.propagate = False
-    return stdout_logger
+    stdout_logger_name = __name__ + "_stdout"
+    stdout_logger = logging.getLogger(stdout_logger_name)
 
+    requires_config = False
+    with configured_loggers_lock:
+        if stdout_logger_name not in configured_loggers:
+            configured_loggers.add(stdout_logger_name)
+            requires_config = True
 
-def __get_logger(debug, name, handler=None):
-    logger = logging.getLogger(name)
-    logger.setLevel(logging.DEBUG if debug else logging.INFO)
-    if handler:
-        logger.addHandler(handler)
-    return logger
+    if requires_config:
+        stdout_logger.propagate = False
+        stdout_logger.setLevel(logging.DEBUG if debug else logging.INFO)
+        stdout_logger.addHandler(logging.StreamHandler(sys.stdout))
+
+    return stdout_logger

requirements.txt

@@ -8,10 +8,10 @@ PyYAML==6.0.1
 requests==2.31.0
 starlette==0.27.0
 werkzeug==2.2.3
+swagger-ui-bundle==0.0.9
 aiohttp_jinja2
 aiohttp
 aiohttp_remotes
 pytest-aiohttp
 starlette
 a2wsgi
-swagger-ui-bundle