Skip to content

Commit 90dccc8

Browse files
committed
fix(node): Allow for undefined transport to be passed in
1 parent d95ef57 commit 90dccc8

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

packages/node/src/sdk/index.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -214,7 +214,6 @@ function getClientOptions(
214214
const tracesSampleRate = getTracesSampleRate(options.tracesSampleRate);
215215

216216
const baseOptions = dropUndefinedKeys({
217-
transport: makeNodeTransport,
218217
dsn: process.env.SENTRY_DSN,
219218
environment: process.env.SENTRY_ENVIRONMENT,
220219
sendClientReports: true,
@@ -223,6 +222,7 @@ function getClientOptions(
223222
const overwriteOptions = dropUndefinedKeys({
224223
release,
225224
tracesSampleRate,
225+
transport: options.transport || makeNodeTransport,
226226
});
227227

228228
const mergedOptions = {

packages/node/test/sdk/init.test.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -211,4 +211,9 @@ describe('validateOpenTelemetrySetup', () => {
211211
expect(errorSpy).toBeCalledWith(expect.stringContaining('You have to set up the SentrySpanProcessor.'));
212212
expect(warnSpy).toBeCalledWith(expect.stringContaining('You have to set up the SentrySampler.'));
213213
});
214+
215+
// Regression test for https://github.com/getsentry/sentry-javascript/issues/15558
216+
it('accepts an undefined transport', () => {
217+
init({ dsn: PUBLIC_DSN, transport: undefined });
218+
});
214219
});

0 commit comments

Comments
 (0)