#32016: mail_admins configuration crashes with ImproperlyConfigured.
-------------------------------------+-------------------------------------
     Reporter:  Susana Cárdenas      |                    Owner:  (none)
  Molinar                            |
         Type:  Bug                  |                   Status:  new
    Component:  Error reporting      |                  Version:  3.1
     Severity:  Release blocker      |               Resolution:
     Keywords:  settings, logging    |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Carlton Gibson):

 Grrr. I think this is a documentation issue. If you configure the logging
 in settings then any depend on *MUST* be defined before you get there
 (because you brought forward the logging config call)

 Here's a PR adding a suggested clarification:
 [https://github.com/django/django/pull/13446 PR]

 The change in 3.1 made MainAdminHandler depend on the settings but I can't
 see it as a bug we should **fix** per se, since manually configuring
 settings, you're kind of on your own.

 I'm not sure I see why you'd do this:


 {{{
 LOGGING_CONFIG = None

 logging.config.dictConfig(...)
 }}}


 By default `LOGGING_CONFIG` just is 'logging.config.dictConfig' 🤔

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

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.9591e42d2051e19f498258aa37ab660f%40djangoproject.com.

Reply via email to