[Django] #24526: Default logging config filters out anything below ERROR for django.request/django.security loggers

14 views
Skip to first unread message

Django

unread,
Mar 23, 2015, 6:08:30 PM3/23/15
to django-...@googlegroups.com
#24526: Default logging config filters out anything below ERROR for
django.request/django.security loggers
-------------------------------------+-------------------------------------
Reporter: timgraham | Owner: timgraham
Type: | Status: new
Cleanup/optimization |
Component: Core | Version: master
(Other) |
Severity: Normal | Keywords:
Triage Stage: | Has patch: 0
Unreviewed |
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
-------------------------------------+-------------------------------------
The fact that the `django.request` and `django.security` handlers define:
{{{
'level': 'ERROR',
'propagate': False,
}}}
makes it impossible to configure other handlers to catch these errors.

--
Ticket URL: <https://code.djangoproject.com/ticket/24526>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

Django

unread,
Mar 26, 2015, 7:53:18 PM3/26/15
to django-...@googlegroups.com
#24526: Default logging config filters out anything below ERROR for
django.request/django.security loggers
-------------------------------------+-------------------------------------
Reporter: timgraham | Owner: timgraham
Type: | Status: new
Cleanup/optimization |
Component: Core (Other) | Version: master
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by timgraham):

* stage: Unreviewed => Accepted


--
Ticket URL: <https://code.djangoproject.com/ticket/24526#comment:1>

Django

unread,
Apr 23, 2015, 1:03:50 PM4/23/15
to django-...@googlegroups.com
#24526: Default logging config filters out anything below ERROR for
django.request/django.security loggers
-------------------------------------+-------------------------------------
Reporter: timgraham | Owner: timgraham
Type: | Status: new
Cleanup/optimization |
Component: Core (Other) | Version: master
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by timgraham):

* has_patch: 0 => 1


Comment:

[https://github.com/django/django/pull/4389 PR] is now ready for review.

--
Ticket URL: <https://code.djangoproject.com/ticket/24526#comment:2>

Django

unread,
Apr 24, 2015, 11:16:10 AM4/24/15
to django-...@googlegroups.com
#24526: Default logging config filters out anything below ERROR for
django.request/django.security loggers
-------------------------------------+-------------------------------------
Reporter: timgraham | Owner: timgraham
Type: | Status: closed

Cleanup/optimization |
Component: Core (Other) | Version: master
Severity: Normal | Resolution: fixed
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Tim Graham <timograham@…>):

* status: new => closed
* resolution: => fixed


Comment:

In [changeset:"8efea1b8d5b5fb0cfef1a3244c339cebf8af36c5" 8efea1b8]:
{{{
#!CommitTicketReference repository=""
revision="8efea1b8d5b5fb0cfef1a3244c339cebf8af36c5"
Fixed #24526 -- Combined django.request/security loggers with the root
logger.

Thanks Carl Meyer for review.
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/24526#comment:3>

Django

unread,
Sep 23, 2015, 11:53:08 AM9/23/15
to django-...@googlegroups.com
#24526: Default logging config filters out anything below ERROR for
django.request/django.security loggers
-------------------------------------+-------------------------------------
Reporter: timgraham | Owner: timgraham
Type: | Status: closed
Cleanup/optimization |
Component: Core (Other) | Version: master
Severity: Normal | Resolution: fixed
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by Tim Graham <timograham@…>):

In [changeset:"6b37719616b77872f9498ca74022563c178031e6" 6b377196]:
{{{
#!CommitTicketReference repository=""
revision="6b37719616b77872f9498ca74022563c178031e6"
Refs #24526 -- Made the django logger handle INFO messages.

Without an explicit 'level', only messages at WARNING or higher
are handled. This makes the config consistent with the docs
which say, "The django catch-all logger sends all messages at
the INFO level or higher to the console."
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/24526#comment:4>

Reply all
Reply to author
Forward
0 new messages