Repository navigation
@sentry/node assert.default is not a function #11002
Description
Is there an existing issue for this?
- I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
- I have reviewed the documentation https://docs.sentry.io/
- I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases
How do you use Sentry?
Sentry Saas (sentry.io)
Which SDK are you using?
@sentry/node
SDK Version
7.160.0
Framework Version
No response
Link to Sentry event
No response
SDK Setup
Sentry.init({
dsn: 'https://d63cc5d3abd141a18ad741ccd49765c3@sentry.5198755.com/32',
environment: process.env.APP_ENV,
tracesSampleRate: 1.0,
release: version,
transportOptions: {
proxy: process.env.HTTPS_PROXY,
},
integrations: (integrations) => {
return integrations.map((integration) => {
if (integration.name === 'OnUncaughtException') {
// override `OnUncaughtException` integration to not exit.
return new Sentry.Integrations.OnUncaughtException({
// do not exit if other uncaught exception handlers are registered.
exitEvenIfOtherHandlersAreRegistered: false,
});
} else {
return integration;
}
});
},
});
Steps to Reproduce
I am currently utilizing a Sentry proxy in my setup. However, this week, I've encountered instability with my proxy network, leading to the issue: assert.default is not a function.
After this error , the process is terminated.
Expected Result
here is souce code: https://github.com/getsentry/sentry-javascript/blob/master/packages/node/src/proxy/index.ts
import * as assert from 'assert';
Actual Result
Metadata
Metadata
Assignees
Projects
- Status
Activity
Lms24 commentedon Mar 11, 2024
Hey @brianYuDesign thanks for writing in!
@timfish when you have a minute, would you mind taking a look at this?
I'm wondering why we have the
assertstatement in the vendored in proxy agent code. Do we need this?https://github.com/getsentry/sentry-javascript/blob/master/packages/node/src/proxy/index.ts#L193
timfish commentedon Mar 11, 2024
I've opened PRs to remove this.
I vendored the code while trying to make as few changes to it as possible and completely missed this!
AbhiPrasad commentedon Mar 14, 2024
Fix released with https://github.com/getsentry/sentry-javascript/releases/tag/7.107.0