Script won't exit with ANR and captureStackTrace active

Is there an existing issue for this?

I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
I have reviewed the documentation https://docs.sentry.io/
I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/node

SDK Version

7.91.0

Framework Version

No response

Link to Sentry event

No response

SDK Setup

 Sentry.init({
    dsn: process.env.SENTRY_DSN,
    integrations: [new Sentry.Integrations.Anr({captureStackTrace: true})],
  });

Steps to Reproduce

import * as Sentry from '@sentry/node';

function configureSentry() {
  Sentry.init({
    dsn: process.env.SENTRY_DSN,
    integrations: [new Sentry.Integrations.Anr({captureStackTrace: true})],
  });
}

async function main() {
  configureSentry();
  await new Promise(resolve => setTimeout(resolve, 1000));
  process.exit(0);
}

main();

Expected Result

The program cleanly exits after 1 second.

Actual Result

The program waits forever with the following console output:

Debugger listening on ws://127.0.0.1:55686/bf184600-b574-49d0-9b00-3640e71b55ed
For help, see: https://nodejs.org/en/docs/inspector
Waiting for the debugger to disconnect...

I can find no workaround.

Thanks for reporting. Sorry, this was unexpected behaviour and therefore didn't even end up in the test cases.

@timfish

Thanks, @timfish . I may have been unclear - #10023 and this issue are separate. #10023 is about unrefing the timer, because "natural" exit doesn't work when ANR is enabled. This issue is about explicit process.exit not working when ANR and captureStackTrace are both enabled.

This issue is about explicit process.exit not working when ANR and captureStackTrace are both enabled.

Ah, thanks for the heads up. I'll test this tomorrow!

I couldn't wait but I'll check again tomorrow because it's late...

With the following code and the timer fix, node exits cleanly after a second:

const Sentry = require('@sentry/node');

function configureSentry() {
  Sentry.init({
    dsn: 'https://public@dsn.ingest.sentry.io/1337',
    integrations: [new Sentry.Integrations.Anr({ captureStackTrace: false })],
  });
}

async function main() {
  configureSentry();
  await new Promise(resolve => setTimeout(resolve, 1000));
  process.exit(0);
}

main();

@timfish

@timfish I apologize; there was a bug in my test case. (That's what I get for trying to do too many things at once...)

I'm seeing the issue with captureStackTrace: true. I've updated my steps to reproduce.

Oh, you're right, it doesn't actually fix it with the timer undef'd. The debugger must keep it alive 😬

All the ANR fixes were released with https://github.com/getsentry/sentry-javascript/releases/tag/7.92.0 - please give it a try!

Thanks for your help + contributions, appreciate it!

Navigation Menu

Repository navigation

Script won't exit with ANR and captureStackTrace active #10022

Is there an existing issue for this?

How do you use Sentry?

Which SDK are you using?

SDK Version

Framework Version

Link to Sentry event

SDK Setup

Steps to Reproduce

Expected Result

Actual Result

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Script won't exit with ANR and captureStackTrace active #10022

Description

Is there an existing issue for this?

How do you use Sentry?

Which SDK are you using?

SDK Version

Framework Version

Link to Sentry event

SDK Setup

Steps to Reproduce

Expected Result

Actual Result

Activity

timfish commented on Jan 2, 2024

joshkel commented on Jan 2, 2024

timfish commented on Jan 2, 2024

timfish commented on Jan 2, 2024

joshkel commented on Jan 2, 2024

timfish commented on Jan 2, 2024

AbhiPrasad commented on Jan 4, 2024

Metadata

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Issue actions