fix(node-core): do not print rejection warning in strict mode

This matches the behavior of Node.js.

Author: mohd-akram

dev-packages/node-core-integration-tests/suites/public-api/onUnhandledRejectionIntegration/mode-strict-error.js

@@ -0,0 +1,14 @@
+const Sentry = require('@sentry/node-core');
+const { setupOtel } = require('../../../utils/setupOtel.js');
+const { expectProcessToExit } = require('../../../utils/expect-process-to-exit');
+
+const client = Sentry.init({
+  dsn: 'https://public@dsn.ingest.sentry.io/1337',
+  integrations: [Sentry.onUnhandledRejectionIntegration({ mode: 'strict' })],
+});
+
+setupOtel(client);
+
+expectProcessToExit();
+
+Promise.reject(new Error('test rejection'));

dev-packages/node-core-integration-tests/suites/public-api/onUnhandledRejectionIntegration/test.ts

@@ -52,8 +52,23 @@ Error: test rejection
         expect(err?.code).toBe(1);
         expect(stdout).not.toBe("I'm alive!");
         expect(stderr)
-          .toContain(`This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). The promise rejected with the reason:
-test rejection`);
+          .toBe('test rejection\n');
+        done();
+      });
+    }));
+
+  test('should show error-type promise rejections in strict mode', () =>
+    new Promise<void>(done => {
+      expect.assertions(4);
+
+      const testScriptPath = path.resolve(__dirname, 'mode-strict-error.js');
+
+      childProcess.execFile('node', [testScriptPath], { encoding: 'utf8' }, (err, stdout, stderr) => {
+        expect(err).not.toBeNull();
+        expect(err?.code).toBe(1);
+        expect(stdout).not.toBe("I'm alive!");
+        expect(stderr)
+          .toContain('Error: test rejection');
         done();
       });
     }));

dev-packages/node-integration-tests/suites/public-api/onUnhandledRejectionIntegration/mode-strict-error.js

@@ -0,0 +1,11 @@
+const Sentry = require('@sentry/node');
+const { expectProcessToExit } = require('../../../utils/expect-process-to-exit');
+
+Sentry.init({
+  dsn: 'https://public@dsn.ingest.sentry.io/1337',
+  integrations: [Sentry.onUnhandledRejectionIntegration({ mode: 'strict' })],
+});
+
+expectProcessToExit();
+
+Promise.reject(new Error('test rejection'));

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

@@ -53,8 +53,23 @@ Error: test rejection
         expect(err?.code).toBe(1);
         expect(stdout).not.toBe("I'm alive!");
         expect(stderr)
-          .toContain(`This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). The promise rejected with the reason:
-test rejection`);
+          .toBe('test rejection\n');
+        done();
+      });
+    }));
+
+  test('should show error-type promise rejections in strict mode', () =>
+    new Promise<void>(done => {
+      expect.assertions(4);
+
+      const testScriptPath = path.resolve(__dirname, 'mode-strict-error.js');
+
+      childProcess.execFile('node', [testScriptPath], { encoding: 'utf8' }, (err, stdout, stderr) => {
+        expect(err).not.toBeNull();
+        expect(err?.code).toBe(1);
+        expect(stdout).not.toBe("I'm alive!");
+        expect(stderr)
+          .toContain('Error: test rejection');
         done();
       });
     }));

packages/node-core/src/integrations/onunhandledrejection.ts

@@ -140,12 +140,9 @@ function handleRejection(reason: unknown, mode: UnhandledRejectionMode): void {
   if (mode === 'warn') {
     consoleSandbox(() => {
       console.warn(rejectionWarning);
-      console.error(reason && typeof reason === 'object' && 'stack' in reason ? reason.stack : reason);
+      console.error(reason);
     });
   } else if (mode === 'strict') {
-    consoleSandbox(() => {
-      console.warn(rejectionWarning);
-    });
     logAndExitProcess(reason);
   }
   /* eslint-enable no-console */