fix(node): Anr should not block exit (#10035)

Author: timfish

dev-packages/node-integration-tests/suites/anr/should-exit.js

@@ -0,0 +1,19 @@
+const Sentry = require('@sentry/node');
+
+function configureSentry() {
+  Sentry.init({
+    dsn: 'https://[email protected]/1337',
+    release: '1.0',
+    debug: true,
+    autoSessionTracking: false,
+    integrations: [new Sentry.Integrations.Anr({ captureStackTrace: true })],
+  });
+}
+
+async function main() {
+  configureSentry();
+  await new Promise(resolve => setTimeout(resolve, 1000));
+  process.exit(0);
+}
+
+main();

dev-packages/node-integration-tests/suites/anr/test.ts

@@ -115,6 +115,20 @@ conditionalTest({ min: 16 })('should report ANR when event loop blocked', () =>
     });
   });
 
+  test('can exit', done => {
+    const testScriptPath = path.resolve(__dirname, 'should-exit.js');
+    let hasClosed = false;
+
+    setTimeout(() => {
+      expect(hasClosed).toBe(true);
+      done();
+    }, 5_000);
+
+    childProcess.exec(`node ${testScriptPath}`, { encoding: 'utf8' }, (_, stdout) => {
+      hasClosed = true;
+    });
+  });
+
   test('With session', done => {
     expect.assertions(9);
 

packages/node/src/integrations/anr/index.ts

@@ -123,6 +123,10 @@ async function _startWorker(client: NodeClient, _options: Partial<Options>): Pro
   // Ensure this thread can't block app exit
   worker.unref();
 
+  process.on('exit', () => {
+    worker.terminate();
+  });
+
   const timer = setInterval(() => {
     try {
       const currentSession = getCurrentScope().getSession();
@@ -135,6 +139,8 @@ async function _startWorker(client: NodeClient, _options: Partial<Options>): Pro
       //
     }
   }, options.pollInterval);
+  // Timer should not block exit
+  timer.unref();
 
   worker.on('message', (msg: string) => {
     if (msg === 'session-ended') {