Tim Fish

Results 386 comments of Tim Fish
trafficstars

Over the weekend I submitted another PR for `import-in-the-middle` which expands `exclude` so that we can use regular expressions: - https://github.com/nodejs/import-in-the-middle/pull/148 In the long run I think we've got a...

If you update your dependencies to ensure that you're using `[email protected]` you can now pass a regular expressions for `exclude`. The Sentry types will be updated in the next release...

The workaround for this issue for now is to exclude openai from being instrumented via the `registerEsmLoaderHooks` init option: ```ts import * as Sentry from '@sentry/node'; Sentry.init({ dsn: '__DSN__', registerEsmLoaderHooks:...

> IIRC we couldn't come up with a solution for the export pattern that openai uses in their package right? Or did I miss something. I seem to remember it...

v8.29.0 of the Node SDK adds the new `onlyIncludeInstrumentedModules` option to `registerEsmLoaderHooks`. ```ts import * as Sentry from '@sentry/node'; Sentry.init({ dsn: '__PUBLIC_DSN__', registerEsmLoaderHooks: { onlyIncludeInstrumentedModules: true }, }); ``` When...

Oh wow, thanks @torickjdavis, that means the release notes are wrong too. I'll correct these!

- Add durable objects instrumentation Durable objects run in a separate isolate with their own `fetch` function. Would you use `withSentry` here? Cloudflare workers also has: - Service Bindings for...

This is currently tracked upstream by https://github.com/DataDog/import-in-the-middle/issues/94 I've been thinking of some ways to work around this and should get a PR submitted this week!

https://github.com/DataDog/import-in-the-middle/pull/115

v8.29.0 of the Node SDK added a new `onlyIncludeInstrumentedModules` option to `registerEsmLoaderHooks`. ```ts import * as Sentry from '@sentry/node'; Sentry.init({ dsn: '__PUBLIC_DSN__', registerEsmLoaderHooks: { onlyIncludeInstrumentedModules: true }, }); ``` When...