Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Deprecate init context manager #3282

Open
szokeasaurusrex opened this issue Jul 12, 2024 · 0 comments · May be fixed by #3729
Open

Deprecate init context manager #3282

szokeasaurusrex opened this issue Jul 12, 2024 · 0 comments · May be fixed by #3729
Assignees
@szokeasaurusrex
Labels
Enhancement New feature or request Triaged Has been looked at recently during old issue triage

Comments

@szokeasaurusrex
Copy link
Member

szokeasaurusrex commented Jul 12, 2024 •
edited
Loading

See below:

init is generally one per process anyway. In case people want DSN isolation, multiple clients is the way to go now. So I really don't see the use case for this.
Let's deprecate and remove.

Originally posted by @sl0thentr0py in #3276 (comment)
@szokeasaurusrex szokeasaurusrex added Triaged Has been looked at recently during old issue triage Enhancement New feature or request labels Nov 4, 2024
@szokeasaurusrex szokeasaurusrex mentioned this issue Nov 4, 2024
Open
szokeasaurusrex added a commit that referenced this issue Nov 5, 2024
@szokeasaurusrex
ref(init): Deprecate sentry_sdk.init context manager
399c344 
It is possible to use the return value of `sentry_sdk.init` as a context manager; however, this functionality has not been maintained for a long time, and it does not seem to be documented anywhere.

So, we are deprecating this functionality, and we will remove it in the next major release.

Closes #3282
@szokeasaurusrex szokeasaurusrex linked a pull request Nov 5, 2024 that will close this issue
Open
@szokeasaurusrex szokeasaurusrex self-assigned this Nov 5, 2024
szokeasaurusrex added a commit that referenced this issue Nov 5, 2024
@szokeasaurusrex
ref(init): Deprecate sentry_sdk.init context manager
fae6713 
It is possible to use the return value of `sentry_sdk.init` as a context manager; however, this functionality has not been maintained for a long time, and it does not seem to be documented anywhere.

So, we are deprecating this functionality, and we will remove it in the next major release.

Closes #3282
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@szokeasaurusrex szokeasaurusrex

Labels
Enhancement New feature or request Triaged Has been looked at recently during old issue triage
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

ref(init): Deprecate sentry_sdk.init context manager getsentry/sentry-python
1 participant
@szokeasaurusrex