-
Notifications
You must be signed in to change notification settings - Fork 29k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Insiders] Debugger kept in invalid state after process exits #124045
Comments
Does the node process terminate properly if you send a SIGINT signal but have no debugger attached? |
Yes, I would have to disable Auto Attach but it does exit properly. And sorry I didn't include the code for the repo, I've edited it and added that. |
Thanks for the issue. This is because you're debugging twice using both auto attach and a launch config. The 'target IDs' are set for the worker by Node, so the first debug session steals the target for the second session (since they're attaching to the same worker), causing an error to be thrown. This prevents the second session from initializing (and being able to be torn down). A workaround is to remove the duplicate session, either:
|
I see. Even with auto attach disabled there was still one debugger open so I couldn't get it work but after restarting the app it worked as expected. Thanks! |
no problem, it should be fixed on nightly |
@a0viedo has the issue been resolved for you (without having to use a workaround)? |
@rzhao271 yes, when I managed to get two debuggers attached I had to restart VS Code and disable auto attach to get it working as expected. |
Does this issue occur when all extensions are disabled?: Yes/No
1.57.0-insider
I've tested this with all extensions disabled.
Steps to Reproduce:
--inspect-brk
that uses worker threadsSIGINT
signal to the process to terminate itEven if I click on disconnect nothing happens and the debugger still will be on screen. The only solution I've found is to force a "Reload Window".
I've uploaded a video of the reproduction too.
Code:
The text was updated successfully, but these errors were encountered: