Django integration hides request handler function name

[thomas-spinergie]

How do you use Sentry?

Sentry Saas (sentry.io)

Version

2.22.0

Steps to Reproduce

The issue seems very similar to this issue solved previously. I am using apache/mod_wsgi which logs internally which functions the threads are running when a crash occurs (like a timeout).

The issue is when inspect the logs, the functions names are the wrapper functions by the Sentry DjangoIntegration and not the original function in my code.

Expected Result

The function name when traced by external libs (like mod_wsgi here) should show the function name handling the Django function view of the route.

Maybe functools.wraps can be used for this purpose like for some other integrations ?

Actual Result

Currently the showed function name is the sentry wrapper names like _before_get_response, sentry_patched_get_response, _set_transaction_name_and_source, sentry_patched_wsgi_handler etc.

[szokeasaurusrex]

Hey @thomas-spinergie, thank you for reporting this. Still need to try and reproduce this problem, but I agree, we should probably use functools.wraps if we are not already using it.

[sentrivana]

Hey @thomas-spinergie, we've now patched more places with @functools.wraps, however since this might lead to breaking changes (e.g. in issue grouping), we've decided to release the change in the next major (probably couple months to go). Just so you know why this is not included in the next regular 2.x release.

[thomas-spinergie]

Nice thanks a lot !