Skip to content
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.

Sign up for GitHub

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

Jump to bottom

isn't SIGUSR1 reserved for the debugger? #40

Open
a0viedo opened this issue Sep 20, 2018 · 2 comments
Open

isn't SIGUSR1 reserved for the debugger? #40

a0viedo opened this issue Sep 20, 2018 · 2 comments
Labels
breaking change bug

Comments

@a0viedo
Copy link

a0viedo commented Sep 20, 2018

In the docs:

'SIGUSR1' is reserved by Node.js to start the debugger. It's possible to install a listener but doing so might interfere with the debugger.

SIGUSR2 should be ok
@yoshuawuyts
Copy link
Contributor

@a0viedo sounds reasonable!

@jonkoops
Copy link
Collaborator

jonkoops commented Jul 9, 2024

This sounds good to me, but considering this would be a breaking change it should be handled in a major version. We can add support for 'SIGUSR2' under this version and then possibly drop SIGUSR1 in the future. Either way, this needs some thought.

@jonkoops jonkoops mentioned this issue Jul 12, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
breaking change bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jonkoops @a0viedo @yoshuawuyts