-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Expose ESM loader exclude
and include
in SDK options
#12878
Labels
Comments
timfish
added
Type: Improvement
Package: node
Issues related to the Sentry Node SDK
labels
Jul 11, 2024
AbhiPrasad
pushed a commit
that referenced
this issue
Jul 15, 2024
Closes #12878 I added this feature to `import-in-the-middle` which was released in v1.9.0: - nodejs/import-in-the-middle#124 This PR changes the hook from `@opentelemetry/instrumentation/hook.mjs` to `import-in-the-middle/hook.mjs` as it was only pasing though anyway and the otel hook doesn't pass the `initialize` export.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Problem Statement
import-in-the-middle
now has a means toexclude
andinclude
libraries which are incompatible.Solution Brainstorm
We should expose these to users via SDK init.
The text was updated successfully, but these errors were encountered: