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
fix: disable SIGUSR1 when --inspect is disabled #33188
Conversation
+ // Controls where or not the InspectorAgent for this Environment should | ||
+ // call StartDebugSignalHandler. This control is needed by embedders who may | ||
+ // not want to allow other processes to start the V8 inspector. | ||
+ kNoStartDebugSignalHandler = 1 << 9 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would it be reasonable to have just one flag for "don't do the inspector stuff please", rather than N flags for each of "don't do this particular inspector thing please"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably but given these flags live in Node and we've already added a few small scoped ones I don't think Node will like the breaking change of modifying the behavior of an existing flag
excited for the next installment of "node added a way to enable the inspector and now we have to add a patch to disable the new way", loved the last season :) |
26cc846
to
d2114cf
Compare
Release Notes Persisted
|
I was unable to backport this PR to "16-x-y" cleanly; |
I was unable to backport this PR to "17-x-y" cleanly; |
I was unable to backport this PR to "18-x-y" cleanly; |
@MarshallOfSound has manually backported this PR to "18-x-y", please check out #34180 |
@MarshallOfSound has manually backported this PR to "17-x-y", please check out #34181 |
@MarshallOfSound has manually backported this PR to "16-x-y", please check out #34182 |
* fix: disable SIGUSR1 when --inspect is disabled (#33188) * Update .patches
fix: disable SIGUSR1 when --inspect is disabled (#33188)
* fix: disable SIGUSR1 when --inspect is disabled (#33188) * Update .patches * chore: rebase patches
As in title / notes
Notes: SIGUSR1 is no longer handled when the
node_cli_inspect
fuse is disabled