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

nodejs · Aug 16, 2021 · 3dc0320 · 3dc0320
1 parent 48c880d
commit 3dc0320
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
@@ -612,6 +612,14 @@ added: v9.0.0
 Disables runtime checks for `async_hooks`. These will still be enabled
 dynamically when `async_hooks` is enabled.
 
+### `--no-global-search-paths`
+<!-- YAML
+added: REPLACEME
+-->
+
+Do not search modules from global paths like `$HOME/.node_modules` and
+`$NODE_PATH`.
+
 ### `--no-warnings`
 <!-- YAML
 added: v6.0.0
@@ -1423,6 +1431,7 @@ Node.js options that are allowed are:
 * `--no-deprecation`
 * `--no-experimental-repl-await`
 * `--no-force-async-hooks-checks`
+* `--no-global-search-paths`
 * `--no-warnings`
 * `--node-memory-debug`
 * `--openssl-config`

diff --git a/doc/node.1 b/doc/node.1
@@ -277,6 +277,9 @@ Silence deprecation warnings.
 Disable runtime checks for `async_hooks`.
 These will still be enabled dynamically when `async_hooks` is enabled.
 .
+.It Fl -no-global-search-paths
+Do not search modules from global paths.
+.
 .It Fl -no-warnings
 Silence all process warnings (including deprecations).
 .

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
@@ -402,6 +402,11 @@ EnvironmentOptionsParser::EnvironmentOptionsParser() {
             &EnvironmentOptions::force_async_hooks_checks,
             kAllowedInEnvironment,
             true);
+  AddOption("--global-search-paths",
+            "disable global module search paths",
+            &EnvironmentOptions::global_search_paths,
+            kAllowedInEnvironment,
+            true);
   AddOption("--warnings",
             "silence all process warnings",
             &EnvironmentOptions::warnings,

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