src: add --no-global-search-paths cli option

nodejs · Aug 16, 2021 · d3ea98b · d3ea98b
1 parent 48c880d
commit d3ea98b
Show file tree

Hide file tree

Showing 5 changed files with 22 additions and 0 deletions.
diff --git a/doc/api/cli.md b/doc/api/cli.md
@@ -619,6 +619,14 @@ added: v6.0.0
 
 Silence all process warnings (including deprecations).
 
+### `--no-global-search-paths`
+<!-- YAML
+added: v16.?.?
+-->
+
+Do not search modules from global paths like `$HOME/.node_modules` and
+`$NODE_PATH`.
+
 ### `--node-memory-debug`
 <!-- YAML
 added: v15.0.0
@@ -1424,6 +1432,7 @@ Node.js options that are allowed are:
 * `--no-experimental-repl-await`
 * `--no-force-async-hooks-checks`
 * `--no-warnings`
+* `--no-global-search-paths`
 * `--node-memory-debug`
 * `--openssl-config`
 * `--pending-deprecation`

diff --git a/doc/node.1 b/doc/node.1
@@ -280,6 +280,9 @@ These will still be enabled dynamically when `async_hooks` is enabled.
 .It Fl -no-warnings
 Silence all process warnings (including deprecations).
 .
+.It Fl -no-global-search-paths
+Do not search modules from global paths.
+.
 .It Fl -node-memory-debug
 Enable extra debug checks for memory leaks in Node.js internals. This is
 usually only useful for developers debugging Node.js itself.

diff --git a/src/env.cc b/src/env.cc
@@ -383,6 +383,10 @@ Environment::Environment(IsolateData* isolate_data,
   inspector_host_port_ = std::make_shared<ExclusiveAccess<HostPort>>(
       options_->debug_options().host_port);
 
+  if (!options_->global_search_paths) {
+    flags_ |= EnvironmentFlags::kNoGlobalSearchPaths;
+  }
+
   if (!(flags_ & EnvironmentFlags::kOwnsProcessState)) {
     set_abort_on_uncaught_exception(false);
   }

diff --git a/src/node_options.cc b/src/node_options.cc
@@ -407,6 +407,11 @@ EnvironmentOptionsParser::EnvironmentOptionsParser() {
             &EnvironmentOptions::warnings,
             kAllowedInEnvironment,
             true);
+  AddOption("--global-search-paths",
+            "disable global module search paths",
+            &EnvironmentOptions::global_search_paths,
+            kAllowedInEnvironment,
+            true);
   AddOption("--force-context-aware",
             "disable loading non-context-aware addons",
             &EnvironmentOptions::force_context_aware,

diff --git a/src/node_options.h b/src/node_options.h
@@ -122,6 +122,7 @@ class EnvironmentOptions : public Options {
   bool deprecation = true;
   bool force_async_hooks_checks = true;
   bool warnings = true;
+  bool global_search_paths = true;
   bool force_context_aware = false;
   bool pending_deprecation = false;
   bool preserve_symlinks = false;