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.

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

Investigate pluggable integrations #13677

Open
9 tasks
mydea opened this issue Sep 12, 2024 · 2 comments
Open
9 tasks

Investigate pluggable integrations #13677

mydea opened this issue Sep 12, 2024 · 2 comments
Labels
Package: core Issues related to the Sentry Core SDK

Comments

@mydea
Copy link
Member

mydea commented Sep 12, 2024

Description

We have a list of non-default integrations in the SDKs. We should take some time to review all of them, and check the following things:

  • What does the integration actually do?
  • Is this valuable? Do we (still) need this?
  • Should this possibly become a default integration?
  • Or else, should this possibly remain pluggable, but recommended in onboarding?
  • Or do we possibly want to remove this in the future because we do not think we need it anymore?

Pluggable integrations from core

  • captureConsoleIntegration
  • debugIntegration
  • extraErrorDataIntegration
  • metadataIntegration
  • sessionTimingIntegration
  • zodErrorsIntegration

Pluggable integrations from browser

  • contextLinesIntegration
  • httpClientIntegration
  • reportingObserverIntegration
@mydea mydea added the Package: core Issues related to the Sentry Core SDK label Sep 12, 2024
@dirstart
Copy link

dirstart commented Sep 25, 2024

I have the same opinion as you.
I also noticed that besides the official repos on GitHub, there aren't any third-party integrations.
I'm looking to build a plugin for white screen detection—should I just go ahead and submit a pull request to the official repo?

@Lms24
Copy link
Member

Lms24 commented Sep 25, 2024

Hey @dirstart 👋 There are some 3rd party SDK integrations for Sentry (that we know of), like for example supabase that live in their own repos. This is perfectly fine for us!

If you want to write an integration for the Sentry SDK, feel free to start out with doing it in your own repo (and npm package). If you then want to contribute the integration to the Sentry repo, feel free to open an issue about it but we can't make guarantees, as this has implications on our support capacities.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Package: core Issues related to the Sentry Core SDK
Projects
Status: No status
Development

No branches or pull requests

3 participants