-
Notifications
You must be signed in to change notification settings - Fork 601
Closed
Labels
Description
How do you use Sentry?
Self-hosted/on-premise
Version
2.28.0
Steps to Reproduce
As I understand the docs, you would disable Loguru integration like follows:
sentry_sdk.init(..., disabled_integrations=[LoguruIntegration()])Expected Result
Loguru integration to be disabled.
Actual Result
LoguruIntegration is enabling itself in its __init__ method, not in setup_once:
sentry-python/sentry_sdk/integrations/loguru.py
Lines 90 to 93 in aa0eaab
| @staticmethod | |
| def setup_once(): | |
| # type: () -> None | |
| pass # we do everything in __init__ |
This means the above code does not actually disable Loguru integration at all, it actually enables it earlier. I don't want to have to disable all automatic integrations just to disable this one.
(Also, given sentry_sdk.init is a context manager, I would expect Loguru integration to be disabled once the context manager was exited -- this would help with running test suites :) )
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Fields
Give feedbackNo fields configured for issues without a type.
Projects
Status
No status