From 6e0565e3c4779964a77de529f418e4aa51c985c6 Mon Sep 17 00:00:00 2001 From: Ahmed Etefy Date: Thu, 6 May 2021 11:08:13 +0200 Subject: [PATCH] Disable autoSessionTracking by default --- packages/node/src/sdk.ts | 2 +- .../single-session/caught-exception-errored-session.js | 1 + .../manual/release-health/single-session/healthy-session.js | 3 ++- .../single-session/uncaught-exception-crashed-session.js | 3 ++- .../single-session/unhandled-rejection-crashed-session.js | 3 ++- 5 files changed, 8 insertions(+), 4 deletions(-) diff --git a/packages/node/src/sdk.ts b/packages/node/src/sdk.ts index 67865c6ea317..13f46a4783a1 100644 --- a/packages/node/src/sdk.ts +++ b/packages/node/src/sdk.ts @@ -104,7 +104,7 @@ export function init(options: NodeOptions = {}): void { } if (options.autoSessionTracking === undefined) { - options.autoSessionTracking = true; + options.autoSessionTracking = false; } if (options.environment === undefined && process.env.SENTRY_ENVIRONMENT) { diff --git a/packages/node/test/manual/release-health/single-session/caught-exception-errored-session.js b/packages/node/test/manual/release-health/single-session/caught-exception-errored-session.js index 7f2bf52a56fa..a8d8a8eb6834 100644 --- a/packages/node/test/manual/release-health/single-session/caught-exception-errored-session.js +++ b/packages/node/test/manual/release-health/single-session/caught-exception-errored-session.js @@ -43,6 +43,7 @@ Sentry.init({ dsn: 'http://test@example.com/1337', release: '1.1', transport: DummyTransport, + autoSessionTracking: true }); /** diff --git a/packages/node/test/manual/release-health/single-session/healthy-session.js b/packages/node/test/manual/release-health/single-session/healthy-session.js index c161963d97c4..8bbb6c6ac3df 100644 --- a/packages/node/test/manual/release-health/single-session/healthy-session.js +++ b/packages/node/test/manual/release-health/single-session/healthy-session.js @@ -32,7 +32,8 @@ class DummyTransport extends BaseDummyTransport { Sentry.init({ dsn: 'http://test@example.com/1337', release: '1.1', - transport: DummyTransport + transport: DummyTransport, + autoSessionTracking: true }); /** diff --git a/packages/node/test/manual/release-health/single-session/uncaught-exception-crashed-session.js b/packages/node/test/manual/release-health/single-session/uncaught-exception-crashed-session.js index 26d22b06a7fd..455da78fa4da 100644 --- a/packages/node/test/manual/release-health/single-session/uncaught-exception-crashed-session.js +++ b/packages/node/test/manual/release-health/single-session/uncaught-exception-crashed-session.js @@ -24,7 +24,8 @@ class DummyTransport extends BaseDummyTransport { Sentry.init({ dsn: 'http://test@example.com/1337', release: '1.1', - transport: DummyTransport + transport: DummyTransport, + autoSessionTracking: true }); /** * The following code snippet will throw an exception of `mechanism.handled` equal to `false`, and so this session diff --git a/packages/node/test/manual/release-health/single-session/unhandled-rejection-crashed-session.js b/packages/node/test/manual/release-health/single-session/unhandled-rejection-crashed-session.js index 1373b6321230..3a472f5e1827 100644 --- a/packages/node/test/manual/release-health/single-session/unhandled-rejection-crashed-session.js +++ b/packages/node/test/manual/release-health/single-session/unhandled-rejection-crashed-session.js @@ -34,7 +34,8 @@ class DummyTransport extends BaseDummyTransport { Sentry.init({ dsn: 'http://test@example.com/1337', release: '1.1', - transport: DummyTransport + transport: DummyTransport, + autoSessionTracking: true }); /**