8000 Deprecate `init` context manager · Issue #3282 · getsentry/sentry-python · GitHub
[go: up one dir, main page]
Skip to content

Deprecate init context manager #3282

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
Closed
szokeasaurusrex opened this issue Jul 12, 2024 · 0 comments · Fixed by #3729
Closed

Deprecate init context manager #3282

szokeasaurusrex opened this issue Jul 12, 2024 · 0 comments · Fixed by #3729
Assignees
@szokeasaurusrex
Labels
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 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 mentioned this issue Nov 5, 2024
Merged
@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
szokeasaurusrex added a commit that referenced this issue Nov 12, 2024
@szokeasaurusrex
ref(init): Deprecate sentry_sdk.init context manager
eba2763 
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
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
0