#32261: Log exceptions handled in Signal.send_robust()
-------------------------------------------------+------------------------
               Reporter:  Adam (Chainz) Johnson  |          Owner:  nobody
                   Type:  New feature            |         Status:  new
              Component:  Core (Other)           |        Version:  master
               Severity:  Normal                 |       Keywords:
           Triage Stage:  Unreviewed             |      Has patch:  0
    Needs documentation:  0                      |    Needs tests:  0
Patch needs improvement:  0                      |  Easy pickings:  0
                  UI/UX:  0                      |
-------------------------------------------------+------------------------
 As pointed out by [https://twitter.com/be_haki/status/1335921247306264579
 Haki Benita on Twitter], by default `Signal.send_robust()` doesn't have
 any log messages for exceptions raised in receivers. Since Django logs
 exceptions in other similar situations, such as missing template
 variables, I think it would be worth adding a `logger.exception()` call in
 the `except` clause of `send_robust()` . Users would then see such
 exceptions in their error handling tools, e.g. Sentry, and be able to
 figure out what action to take from there. Ultimately any *expected*
 exception should be caught with a `try` in the receiver function.

-- 
Ticket URL: <https://code.djangoproject.com/ticket/32261>
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 [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/053.628c01943c3cffb5c086142271b8dd84%40djangoproject.com.

Reply via email to