-
Notifications
You must be signed in to change notification settings - Fork 541
Add include_source_context
option
#2017
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Labels
Good first issue
Good for newcomers
Comments
farhat-nawaz
pushed a commit
to farhat-nawaz/sentry-python
that referenced
this issue
Apr 17, 2023
Some users do not like the source context to be there, and so add `include_source_context` option to opt-out. Fixes getsentry#2017
As of right now this can be removed in
|
Reopening since there's a bit missing in order to finish the implementation fully. |
This was referenced May 11, 2023
Documentation for this option is still missing, but the option itself is already release in version |
documentation PR is on the way: getsentry/sentry-docs#7090 So we can close this. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Problem Statement
Some users do not like the source context to be there and want to disable it.
Solution Brainstorm
For this we need to add a SDK option
include_source_context
that defaults toTrue
.If set to false, the source context should not be included.
Here is probably the place where to decide whether to add the source context or not:
https://github.com/getsentry/sentry-python/blob/antonpirker/celery-beat-wo-restart/sentry_sdk/utils.py#L614-L625
The text was updated successfully, but these errors were encountered: