Making Sentry report 5XX errors in multi-thread environment

Question

I am using Sentry to power up exception handling logging in my app.

The issue arises in the following code snippet:

@api_view(['POST'])
def testView(request):
    a = 1/0 # This error is reported to Sentry
    TestThread().start()
    return f_response_ok()

class TestThread(threading.Thread):

    def __init__(self, *args, **kwargs):
        super(TestThread, self).__init__(*args, **kwargs)

    def run(self):
        print('Test')
        a = 1/0 # but this one is not
        return True

Is it possible to make Sentry report errors that have occurred in parallel thread?

And a bit off-topic: I would appreciate if someone provides a short comment as to whether such programming pattern is obsolete (and stuff like RabbitMQ should be used instead).

I've updated the topic of the question. I mean, 5XX errors - that is, anything — Edgar Navasardyan, Jun 05 '18 at 14:42
Perhaps you are using Django's default 500 handler instead of Sentry's one. Do you mind sharing relevant `settings.py` entries? — Paulo Scardine, Jun 05 '18 at 14:45
Well, in the main thread, errors are reported. See the updated code snippet — Edgar Navasardyan, Jun 05 '18 at 14:46
@EdgarNavasardyan this question is kinda old, but it really seems like you want Celery or something like that. I can't imagine that spawning a thread inside of a WSGI app goes well. Raven has integration for Celery. — Markus Unterwaditzer, Aug 21 '18 at 20:40

score 3 · Accepted Answer · answered Jun 05 '18 at 14:47

You could manually log them to sentry.

https://docs.sentry.io/clients/python/#capture-an-error

Assuming you are using django

from raven.contrib.django.raven_compat.models import client

class TestThread(threading.Thread):

    def __init__(self, *args, **kwargs):
        super(TestThread, self).__init__(*args, **kwargs)

    def run(self):
        print('Test')
        try:
            a = 1/0 # error is not reported in Sentry
        except: # I would suggest putting here expected exceptions
            client.captureException()
        return True

This is extremely useful hint, Sardorbek. You solved one more related issue for me. Thanks !!! I will wait for a while. And unless I get more generic answer, will accept this one as solution — Edgar Navasardyan, Jun 05 '18 at 14:51

Making Sentry report 5XX errors in multi-thread environment

1 Answers1