From 796167aab0395232d9aa9a4050589e941017f18f Mon Sep 17 00:00:00 2001 From: Benjamin Dauvergne Date: Thu, 12 Jan 2012 14:50:17 +0100 Subject: [PATCH] [logging] fix wrong reuse of the auth_logger global creating an infinite chain of objects --- docbow_project/docbow/signals.py | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/docbow_project/docbow/signals.py b/docbow_project/docbow/signals.py index 84f3d70..baafea7 100644 --- a/docbow_project/docbow/signals.py +++ b/docbow_project/docbow/signals.py @@ -14,13 +14,13 @@ auth_logger = logging.getLogger('docbow.auth') def logged_in_handler(sender, request, user, **kwargs): global auth_logger - auth_logger = DjangoLoggerAdapter(auth_logger, request) - auth_logger.info(_('logged in')) + new_logger = DjangoLoggerAdapter(auth_logger, request) + new_logger.info(_('logged in')) def logged_out_handler(sender, request, user, **kwargs): global auth_logger - auth_logger = DjangoLoggerAdapter(auth_logger, request) - auth_logger.info(_('logged out')) + new_logger = DjangoLoggerAdapter(auth_logger, request) + new_logger.info(_('logged out')) def modified_data(sender, instance, created, raw, using, **kwargs): global db_logger @@ -35,6 +35,7 @@ def modified_data(sender, instance, created, raw, using, **kwargs): name=instance), extra=extra) def list_m2m_changed_handler(sender, instance, action, reverse, model, pk_set, using, **kwargs): + global db_logger msg = None if action == 'post_add': msg = _('added user %(user)s to list %(list)s')