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

Sentry dependency issue #100

Open
anuress opened this issue Apr 7, 2023 · 1 comment
Open

Sentry dependency issue #100

anuress opened this issue Apr 7, 2023 · 1 comment

Comments

@anuress
Copy link

anuress commented Apr 7, 2023

Hi, currently I'm integrating xendit-sdk and found some issues with Sentry (the project used Sentry too). From what I saw in the code, won't the way xendit-sdk initializing Sentry will replace our own Sentry initialization?

@freszu
Copy link

freszu commented Sep 7, 2023

Came across this issue while investigating possible integration in one of my projects but its a showstopper. Sentry sdk is not intended for Libraries which is noted in Sentry documentation - is xendit going to do anything about it? :

It is bad practice to use Sentry SDKs in third-party plugins, packages, and libraries (such as payment SDKs or embeddable widgets) meant to be consumed by other apps. If an app uses a Sentry SDK, as well as a third-party package with its own Sentry SDK installed, then the following issues can occur:

  • Version conflicts.
  • Signal handlers may overwrite each other, leading to one of the Sentry SDKs operating incorrectly.
  • Third-party SDKs and libraries may leak users' PII data to the third-party's Sentry organization.

source: https://docs.sentry.io/platforms/

see also: getsentry/sentry-docs#3611

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants