feat(core)!: Enable logs by default

dev-packages/e2e-tests/test-applications/deno-streamed/src/app.ts

@@ -22,7 +22,6 @@ Sentry.init({
   traceLifecycle: 'stream',
   tracesSampleRate: 1,
   dataCollection: { userInfo: true },
-  enableLogs: true,
 });
 
 const port = 3030;

dev-packages/e2e-tests/test-applications/deno/src/app.ts

@@ -24,7 +24,6 @@ Sentry.init({
   tunnel: 'http://localhost:3031/',
   tracesSampleRate: 1,
   dataCollection: { userInfo: true },
-  enableLogs: true,
 });
 
 const port = 3030;

dev-packages/e2e-tests/test-applications/effect-3-browser/src/index.js

@@ -19,7 +19,6 @@ const AppLayer = Layer.mergeAll(
     release: 'e2e-test',
     environment: 'qa',
     tunnel: 'http://localhost:3031',
-    enableLogs: true,
   }),
   Layer.setTracer(Sentry.SentryEffectTracer),
   Logger.replace(Logger.defaultLogger, Sentry.SentryEffectLogger),

dev-packages/e2e-tests/test-applications/effect-3-node/src/app.ts

@@ -15,7 +15,6 @@ const SentryLive = Layer.mergeAll(
     debug: !!process.env.DEBUG,
     tunnel: 'http://localhost:3031/',
     tracesSampleRate: 1,
-    enableLogs: true,
   }),
   Layer.setTracer(Sentry.SentryEffectTracer),
   Logger.replace(Logger.defaultLogger, Sentry.SentryEffectLogger),

dev-packages/e2e-tests/test-applications/effect-4-browser/src/index.js

@@ -19,7 +19,6 @@ const AppLayer = Layer.mergeAll(
     release: 'e2e-test',
     environment: 'qa',
     tunnel: 'http://localhost:3031',
-    enableLogs: true,
   }),
   Logger.layer([Sentry.SentryEffectLogger]),
   Layer.succeed(Tracer.Tracer, Sentry.SentryEffectTracer),

dev-packages/e2e-tests/test-applications/effect-4-node/src/app.ts

@@ -16,7 +16,6 @@ const SentryLive = Layer.mergeAll(
     debug: !!process.env.DEBUG,
     tunnel: 'http://localhost:3031/',
     tracesSampleRate: 1,
-    enableLogs: true,
   }),
   Logger.layer([Sentry.SentryEffectLogger]),
   Layer.succeed(Tracer.Tracer, Sentry.SentryEffectTracer),

dev-packages/e2e-tests/test-applications/node-express-streaming/src/app.ts

@@ -7,7 +7,6 @@ Sentry.init({
   debug: !!process.env.DEBUG,
   tunnel: `http://localhost:3031/`, // proxy server
   tracesSampleRate: 1,
-  enableLogs: true,
   traceLifecycle: 'stream',
   integrations: [
     Sentry.spanStreamingIntegration(),

dev-packages/e2e-tests/test-applications/node-express-v5/src/app.ts

@@ -13,7 +13,6 @@ Sentry.init({
   debug: !!process.env.DEBUG,
   tunnel: `http://localhost:3031/`, // proxy server
   tracesSampleRate: 1,
-  enableLogs: true,
   integrations: [Sentry.nodeRuntimeMetricsIntegration({ collectionIntervalMs: 1_000 })],
 });
 

dev-packages/e2e-tests/test-applications/node-express/src/app.ts

@@ -13,7 +13,6 @@ Sentry.init({
   debug: !!process.env.DEBUG,
   tunnel: `http://localhost:3031/`, // proxy server
   tracesSampleRate: 1,
-  enableLogs: true,
   integrations: [
     Sentry.nativeNodeFetchIntegration({
       headersToSpanAttributes: {

dev-packages/e2e-tests/test-applications/node-otel-without-tracing/src/instrument.ts

@@ -17,7 +17,6 @@ Sentry.init({
   // Tracing is completely disabled
   // Custom OTEL setup
   skipOpenTelemetrySetup: true,
-  enableLogs: true,
 });
 
 // Create and configure NodeTracerProvider

dev-packages/e2e-tests/test-applications/tsx-express/instrument.mjs

@@ -6,5 +6,4 @@ Sentry.init({
   debug: !!process.env.DEBUG,
   tunnel: `http://localhost:3031/`, // proxy server
   tracesSampleRate: 1,
-  enableLogs: true,
 });

dev-packages/node-integration-tests/suites/consola/instrument.mjs

@@ -5,6 +5,5 @@ Sentry.init({
   dsn: 'https://public@dsn.ingest.sentry.io/1337',
   release: '1.0.0',
   environment: 'test',
-  enableLogs: true,
   transport: loggingTransport,
 });

dev-packages/node-integration-tests/suites/pino/instrument-auto-off.mjs

@@ -4,6 +4,5 @@ Sentry.init({
   dsn: process.env.SENTRY_DSN,
   release: '1.0',
   tracesSampleRate: 1.0,
-  enableLogs: true,
   integrations: [Sentry.pinoIntegration({ autoInstrument: false })],
 });

dev-packages/node-integration-tests/suites/pino/instrument.mjs

@@ -4,6 +4,5 @@ Sentry.init({
   dsn: process.env.SENTRY_DSN,
   release: '1.0',
   tracesSampleRate: 1.0,
-  enableLogs: true,
   integrations: [Sentry.pinoIntegration({ error: { levels: ['error', 'fatal'] } })],
 });

dev-packages/node-integration-tests/suites/public-api/logger/scenario.ts

@@ -5,7 +5,6 @@ Sentry.init({
   dsn: 'https://public@dsn.ingest.sentry.io/1337',
   release: '1.0.0',
   environment: 'test',
-  enableLogs: true,
   transport: loggingTransport,
 });
 

dev-packages/node-integration-tests/suites/public-api/logs-disabled/subject.ts

@@ -0,0 +1,31 @@
+import * as Sentry from '@sentry/node';
+import { loggingTransport } from '@sentry-internal/node-integration-tests';
+
+const client = new Sentry.NodeClient({
+  dsn: 'https://public@dsn.ingest.sentry.io/1337',
+  enableLogs: false,
+  transport: loggingTransport,
+  stackParser: Sentry.defaultStackParser,
+  integrations: [],
+});
+
+const scope = new Sentry.Scope();
+scope.setClient(client);
+client.init();
+
+async function run(): Promise<void> {
+  Sentry.logger.info('this log should not be captured', {}, { scope });
+
+  // Flush the log buffer before the sentinel is captured. If the disable path is
+  // broken, the leaked log envelope is sent here and arrives before the error,
+  // failing the ordered `event` expectation. If logs are correctly disabled,
+  // the buffer is empty and only the sentinel error is delivered.
+  await client.flush();
+
+  scope.captureException(new Error('sentinel_error'));
+
+  await client.flush();
+}
+
+// eslint-disable-next-line @typescript-eslint/no-floating-promises
+void run();

dev-packages/node-integration-tests/suites/public-api/logs-disabled/test.ts

@@ -0,0 +1,30 @@
+import { afterAll, describe, test } from 'vitest';
+import { cleanupChildProcesses, createRunner } from '../../../utils/runner';
+
+describe('logs disabled', () => {
+  afterAll(() => {
+    cleanupChildProcesses();
+  });
+
+  // With `enableLogs: false` the log is dropped at capture time, so it never reaches
+  // the transport. The sentinel error is the only envelope we expect — if a log
+  // envelope were emitted, it would arrive before the error and fail the assertion.
+  test('does not capture logs when enableLogs is disabled', async () => {
+    const runner = createRunner(__dirname, 'subject.ts')
+      .expect({
+        event: {
+          exception: {
+            values: [
+              {
+                type: 'Error',
+                value: 'sentinel_error',
+              },
+            ],
+          },
+        },
+      })
+      .start();
+
+    await runner.completed();
+  });
+});

dev-packages/node-integration-tests/suites/public-api/logs/subject.ts

@@ -6,7 +6,6 @@ const client = new Sentry.NodeClient({
   transport: loggingTransport,
   stackParser: Sentry.defaultStackParser,
   integrations: [],
-  enableLogs: true,
   dataCollection: { userInfo: true },
 });
 

dev-packages/node-integration-tests/suites/winston/instrument.mjs

@@ -5,7 +5,6 @@ Sentry.init({
   dsn: 'https://public@dsn.ingest.sentry.io/1337',
   release: '1.0.0',
   environment: 'test',
-  enableLogs: true,
   transport: loggingTransport,
   debug: true,
 });

packages/core/src/client.ts

@@ -265,9 +265,9 @@ export abstract class Client<O extends ClientOptions = ClientOptions> {
     }
 
     // Backfill enableLogs option from _experiments.enableLogs
-    // TODO(v11): Remove or change default value
+    // todo(v11): Remove the experimental flag
     // eslint-disable-next-line typescript/no-deprecated
-    this._options.enableLogs = this._options.enableLogs ?? this._options._experiments?.enableLogs;
+    this._options.enableLogs = this._options.enableLogs ?? this._options._experiments?.enableLogs ?? true;
 
     // Setup log flushing with weight and timeout tracking
     if (this._options.enableLogs) {

packages/core/src/logs/internal.ts

@@ -84,7 +84,7 @@ export function _INTERNAL_captureLog(
     return;
   }
 
-  const { release, environment, enableLogs = false, beforeSendLog } = client.getOptions();
+  const { release, environment, enableLogs = true, beforeSendLog } = client.getOptions();
   if (!enableLogs) {
     DEBUG_BUILD && debug.warn('logging option not enabled, log will not be captured.');
     return;

packages/core/src/types/options.ts

@@ -583,7 +583,7 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
   /**
    * If logs support should be enabled.
    *
-   * @default false
+   * @default true
    */
   enableLogs?: boolean;
 

packages/core/test/lib/client.test.ts

@@ -103,7 +103,7 @@ describe('Client', () => {
       const options = getDefaultTestClientOptions({ dsn: PUBLIC_DSN, test: true });
       const client = new TestClient(options);
 
-      expect(client.getOptions()).toEqual({ attachStacktrace: true, ...options });
+      expect(client.getOptions()).toEqual({ attachStacktrace: true, ...options, enableLogs: true });
     });
   });
 
@@ -2242,7 +2242,7 @@ describe('Client', () => {
         .spyOn(logsInternalModule, '_INTERNAL_flushLogsBuffer')
         .mockImplementation(() => undefined);
 
-      const options = getDefaultTestClientOptions({ dsn: PUBLIC_DSN, enableLogs: true });
+      const options = getDefaultTestClientOptions({ dsn: PUBLIC_DSN });
       const client = new TestClient(options);
 
       await client.close();
@@ -2924,32 +2924,32 @@ describe('Client', () => {
   });
 
   describe('enableLogs', () => {
-    it('defaults to  `undefined`', () => {
+    it('defaults to `true`', () => {
       const options = getDefaultTestClientOptions({ dsn: PUBLIC_DSN });
       const client = new TestClient(options);
-      expect(client.getOptions().enableLogs).toBeUndefined();
+      expect(client.getOptions().enableLogs).toBe(true);
     });
 
-    it('can be set as a top-level option', () => {
-      const options = getDefaultTestClientOptions({ dsn: PUBLIC_DSN, enableLogs: true });
+    it('can be disabled via the top-level option', () => {
+      const options = getDefaultTestClientOptions({ dsn: PUBLIC_DSN, enableLogs: false });
       const client = new TestClient(options);
-      expect(client.getOptions().enableLogs).toBe(true);
+      expect(client.getOptions().enableLogs).toBe(false);
     });
 
-    it('can be set as an experimental option', () => {
-      const options = getDefaultTestClientOptions({ dsn: PUBLIC_DSN, _experiments: { enableLogs: true } });
+    it('can be disabled via the experimental option', () => {
+      const options = getDefaultTestClientOptions({ dsn: PUBLIC_DSN, _experiments: { enableLogs: false } });
       const client = new TestClient(options);
-      expect(client.getOptions().enableLogs).toBe(true);
+      expect(client.getOptions().enableLogs).toBe(false);
     });
 
     test('top-level option takes precedence over experimental option', () => {
       const options = getDefaultTestClientOptions({
         dsn: PUBLIC_DSN,
-        enableLogs: true,
-        _experiments: { enableLogs: false },
+        enableLogs: false,
+        _experiments: { enableLogs: true },
       });
       const client = new TestClient(options);
-      expect(client.getOptions().enableLogs).toBe(true);
+      expect(client.getOptions().enableLogs).toBe(false);
     });
   });
 
@@ -2965,7 +2965,6 @@ describe('Client', () => {
     it('flushes logs when weight exceeds 800KB', () => {
       const options = getDefaultTestClientOptions({
         dsn: PUBLIC_DSN,
-        enableLogs: true,
       });
       const client = new TestClient(options);
       const scope = new Scope();
@@ -2983,7 +2982,6 @@ describe('Client', () => {
     it('accumulates log weight without flushing when under threshold', () => {
       const options = getDefaultTestClientOptions({
         dsn: PUBLIC_DSN,
-        enableLogs: true,
       });
       const client = new TestClient(options);
       const scope = new Scope();
@@ -3001,7 +2999,6 @@ describe('Client', () => {
     it('flushes logs after idle timeout', () => {
       const options = getDefaultTestClientOptions({
         dsn: PUBLIC_DSN,
-        enableLogs: true,
       });
       const client = new TestClient(options);
       const scope = new Scope();
@@ -3023,7 +3020,6 @@ describe('Client', () => {
     it('does not reset idle timeout when new logs are captured', () => {
       const options = getDefaultTestClientOptions({
         dsn: PUBLIC_DSN,
-        enableLogs: true,
       });
       const client = new TestClient(options);
       const scope = new Scope();
@@ -3050,7 +3046,6 @@ describe('Client', () => {
     it('starts new timer after timeout completes and flushes', () => {
       const options = getDefaultTestClientOptions({
         dsn: PUBLIC_DSN,
-        enableLogs: true,
       });
       const client = new TestClient(options);
       const scope = new Scope();
@@ -3082,7 +3077,6 @@ describe('Client', () => {
     it('flushes logs on flush event', () => {
       const options = getDefaultTestClientOptions({
         dsn: PUBLIC_DSN,
-        enableLogs: true,
       });
       const client = new TestClient(options);
       const scope = new Scope();
@@ -3103,6 +3097,7 @@ describe('Client', () => {
     it('does not flush logs when logs are disabled', () => {
       const options = getDefaultTestClientOptions({
         dsn: PUBLIC_DSN,
+        enableLogs: false,
       });
       const client = new TestClient(options);
       const scope = new Scope();
@@ -3122,7 +3117,6 @@ describe('Client', () => {
 
       const options = getDefaultTestClientOptions({
         dsn: PUBLIC_DSN,
-        enableLogs: true,
       });
       const client = new TestClient(options);
       const scope = new Scope();
@@ -3140,9 +3134,7 @@ describe('Client', () => {
 
     it('flush() drains the log buffer when client has no transport', async () => {
       // Client without DSN — _transport is undefined
-      const options = getDefaultTestClientOptions({
-        enableLogs: true,
-      });
+      const options = getDefaultTestClientOptions({});
       const client = new TestClient(options);
       const scope = new Scope();
       scope.setClient(client);

packages/core/test/lib/integrations/consola.test.ts

@@ -32,10 +32,8 @@ describe('createConsolaReporter', () => {
   beforeEach(() => {
     vi.clearAllMocks();
 
-    // Create a test client with enableLogs: true
     mockClient = new TestClient({
       ...getDefaultTestClientOptions({ dsn: 'https://username@domain/123' }),
-      enableLogs: true,
       normalizeDepth: 3,
       normalizeMaxBreadth: 1000,
     });

packages/core/test/lib/logs/console-integration.test.ts

@@ -44,7 +44,6 @@ describe('consoleLoggingIntegration', () => {
 
     client = new TestClient({
       ...getDefaultTestClientOptions({ dsn: 'https://username@domain/123' }),
-      enableLogs: true,
       normalizeDepth: 3,
       normalizeMaxBreadth: 1000,
     });
@@ -418,7 +417,6 @@ describe('consoleLoggingIntegration', () => {
     it('only captures configured levels', () => {
       const filteredClient = new TestClient({
         ...getDefaultTestClientOptions({ dsn: 'https://username@domain/123' }),
-        enableLogs: true,
       });
       vi.mocked(getClient).mockReturnValue(filteredClient);