From 4fd846a84d7949b1e8046366cec12678e1dfe90b Mon Sep 17 00:00:00 2001 From: James O'Beirne Date: Fri, 22 Jan 2016 16:50:50 -0800 Subject: [PATCH 1/2] Remove unnecessary/confusing logger variable in celery --- raven/contrib/celery/__init__.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/raven/contrib/celery/__init__.py b/raven/contrib/celery/__init__.py index 56b72474..7cf250e4 100644 --- a/raven/contrib/celery/__init__.py +++ b/raven/contrib/celery/__init__.py @@ -7,7 +7,6 @@ raven.contrib.celery """ from __future__ import absolute_import -import logging from celery.signals import after_setup_logger, task_failure from raven.handlers.logging import SentryHandler @@ -39,8 +38,6 @@ def register_signal(client): def register_logger_signal(client, logger=None, loglevel=logging.ERROR): filter_ = CeleryFilter() - if logger is None: - logger = logging.getLogger() handler = SentryHandler(client) handler.setLevel(loglevel) handler.addFilter(filter_) From d0adbbf2084ca6220bdd208d3994f282890332cd Mon Sep 17 00:00:00 2001 From: James O'Beirne Date: Fri, 22 Jan 2016 17:44:10 -0800 Subject: [PATCH 2/2] Update __init__.py --- raven/contrib/celery/__init__.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/raven/contrib/celery/__init__.py b/raven/contrib/celery/__init__.py index 7cf250e4..ba6186e9 100644 --- a/raven/contrib/celery/__init__.py +++ b/raven/contrib/celery/__init__.py @@ -7,6 +7,8 @@ raven.contrib.celery """ from __future__ import absolute_import +import logging + from celery.signals import after_setup_logger, task_failure from raven.handlers.logging import SentryHandler