-
Notifications
You must be signed in to change notification settings - Fork 15k
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
feat: add fuses for NODE_OPTIONS and --inspect #30190
Conversation
a72e145
to
7a89373
Compare
patches/node/add_should_read_node_options_from_env_option_to_disable_node_options.patch
Show resolved
Hide resolved
patches/node/add_should_read_node_options_from_env_option_to_disable_node_options.patch
Outdated
Show resolved
Hide resolved
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.
Looks reasonable. I had a few nits, but only nits.
+1 on @nornagon's comment about offering it to upstream.
patches/node/add_should_read_node_options_from_env_option_to_disable_node_options.patch
Outdated
Show resolved
Hide resolved
Release Notes Persisted
|
I was unable to backport this PR to "14-x-y" cleanly; |
I have automatically backported this PR to "15-x-y", please check out #30420 |
* feat: add fuses for NODE_OPTIONS and --inspect * chore: add node patch to ensure NODE_OPTIONS are never parsed when fuse is disabledd * chore: fix lint * chore: flip boolean logic * chore: update patches * chore: add trailing _ to static member * Update add_should_read_node_options_from_env_option_to_disable_node_options.patch * chore: update patches Co-authored-by: PatchUp <73610968+patchup[bot]@users.noreply.github.com>
As in title / notes. Allows apps to completely opt out of these arguments / env vars in packaged apps.
Notes: Added fuses to disable
NODE_OPTIONS
and--inspect
debug arguments in packaged apps