Fix loggers duplication

.gitignore

@@ -10,3 +10,4 @@ dist/
 poetry.lock
 .vscode
 .python-version
+.idea

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## [v2.2.2] - 2023-06-23
+
+- Fix loggers duplication https://github.com/litl/backoff/issues/179
+
 ## [v2.2.1] - 2022-10-05
 
 - Fix type hint for wait generators https://github.com/litl/backoff/issues/177

backoff/_decorator.py

@@ -80,24 +80,22 @@ def on_predicate(wait_gen: _WaitGenerator,
             args will first be evaluated and their return values passed.
             This is useful for runtime configuration.
     """
-    def decorate(target):
-        nonlocal logger, on_success, on_backoff, on_giveup
-
-        logger = _prepare_logger(logger)
-        on_success = _config_handlers(on_success)
-        on_backoff = _config_handlers(
-            on_backoff,
-            default_handler=_log_backoff,
-            logger=logger,
-            log_level=backoff_log_level
-        )
-        on_giveup = _config_handlers(
-            on_giveup,
-            default_handler=_log_giveup,
-            logger=logger,
-            log_level=giveup_log_level
-        )
+    logger = _prepare_logger(logger)
+    on_success = _config_handlers(on_success)
+    on_backoff = _config_handlers(
+        on_backoff,
+        default_handler=_log_backoff,
+        logger=logger,
+        log_level=backoff_log_level
+    )
+    on_giveup = _config_handlers(
+        on_giveup,
+        default_handler=_log_giveup,
+        logger=logger,
+        log_level=giveup_log_level
+    )
 
+    def decorate(target):
         if asyncio.iscoroutinefunction(target):
             retry = _async.retry_predicate
         else:
@@ -180,24 +178,22 @@ def on_exception(wait_gen: _WaitGenerator,
             args will first be evaluated and their return values passed.
             This is useful for runtime configuration.
     """
-    def decorate(target):
-        nonlocal logger, on_success, on_backoff, on_giveup
-
-        logger = _prepare_logger(logger)
-        on_success = _config_handlers(on_success)
-        on_backoff = _config_handlers(
-            on_backoff,
-            default_handler=_log_backoff,
-            logger=logger,
-            log_level=backoff_log_level,
-        )
-        on_giveup = _config_handlers(
-            on_giveup,
-            default_handler=_log_giveup,
-            logger=logger,
-            log_level=giveup_log_level,
-        )
+    logger = _prepare_logger(logger)
+    on_success = _config_handlers(on_success)
+    on_backoff = _config_handlers(
+        on_backoff,
+        default_handler=_log_backoff,
+        logger=logger,
+        log_level=backoff_log_level,
+    )
+    on_giveup = _config_handlers(
+        on_giveup,
+        default_handler=_log_giveup,
+        logger=logger,
+        log_level=giveup_log_level,
+    )
 
+    def decorate(target):
         if asyncio.iscoroutinefunction(target):
             retry = _async.retry_exception
         else: