Fix the streamhandler bug

Author: emmuhamm

src/daqpytools/apps/logging_demonstrator.py

@@ -42,6 +42,10 @@ def test_main_functions(main_logger):
     main_logger.debug("example debug message")
     main_logger.info("example info message")
     main_logger.warning("example warning message")
+    
+    # Error and critical will print twice when using stream handlers
+    # This is because StreamHandlers add in a stream each for stdout and stderr
+    # stderr log level set to error, so at error or above both handlers will fire
     main_logger.error("example error message")
     main_logger.critical("example critical message")
     main_logger.info(

src/daqpytools/logging/logger.py

@@ -1,6 +1,7 @@
 import logging
 from logging import PlaceHolder
 
+import sys
 import kafka
 import sh
 from rich.traceback import install as rich_traceback_install
@@ -159,7 +160,7 @@ def get_daq_logger(
     if log_level is not logging.NOTSET:
         for handler in logger.handlers:
             # Ignore stderr handler resets, needs to be fixed at the error level
-            if type(handler).__name__ == "StderrHandler":
+            if isinstance(handler, logging.StreamHandler) and handler.stream == sys.stderr:
                 continue
             handler.setLevel(log_level)