From 98bba4cc7798f3b8ccd0f1a6d6bc83082d05fdf2 Mon Sep 17 00:00:00 2001 From: heitorlessa Date: Mon, 22 Jul 2024 11:02:27 +0200 Subject: [PATCH] chore(logger): use package logger over source logger to reduce noise --- aws_lambda_powertools/logging/utils.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/aws_lambda_powertools/logging/utils.py b/aws_lambda_powertools/logging/utils.py index 3e1c3c69aed..45819322c28 100644 --- a/aws_lambda_powertools/logging/utils.py +++ b/aws_lambda_powertools/logging/utils.py @@ -4,6 +4,7 @@ from .logger import Logger PACKAGE_LOGGER = "aws_lambda_powertools" +LOGGER = logging.getLogger(__name__) def copy_config_to_registered_loggers( @@ -56,7 +57,7 @@ def copy_config_to_registered_loggers( loggers = exclude filter_func = _exclude_registered_loggers_filter - registered_loggers = _find_registered_loggers(source_logger, loggers, filter_func) + registered_loggers = _find_registered_loggers(loggers=loggers, filter_func=filter_func) for logger in registered_loggers: _configure_logger(source_logger=source_logger, logger=logger, level=level, ignore_log_level=ignore_log_level) @@ -72,13 +73,12 @@ def _exclude_registered_loggers_filter(loggers: Set[str]) -> List[logging.Logger def _find_registered_loggers( - source_logger: Logger, loggers: Set[str], filter_func: Callable[[Set[str]], List[logging.Logger]], ) -> List[logging.Logger]: """Filter root loggers based on provided parameters.""" root_loggers = filter_func(loggers) - source_logger.debug(f"Filtered root loggers: {root_loggers}") + LOGGER.debug(f"Filtered root loggers: {root_loggers}") return root_loggers @@ -91,7 +91,7 @@ def _configure_logger( # customers may not want to copy the same log level from Logger to discovered loggers if not ignore_log_level: logger.setLevel(level) - source_logger.debug(f"Logger {logger} reconfigured to use logging level {level}") + LOGGER.debug(f"Logger {logger} reconfigured to use logging level {level}") logger.handlers = [] logger.propagate = False # ensure we don't propagate logs to existing loggers, #1073 @@ -99,4 +99,4 @@ def _configure_logger( for source_handler in source_logger.handlers: logger.addHandler(source_handler) - source_logger.debug(f"Logger {logger} reconfigured to use {source_handler}") + LOGGER.debug(f"Logger {logger} reconfigured to use {source_handler}")