From 9a672961fafd896f68a39e2d55f744a625417a4d Mon Sep 17 00:00:00 2001
From: Cheng Zhao <zcbenz@gmail.com>
Date: Mon, 16 Aug 2021 09:58:58 +0900
Subject: [PATCH] src: add --no-global-search-paths cli option
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

PR-URL: https://github.com/nodejs/node/pull/39754
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
---
 doc/api/cli.md      | 9 +++++++++
 doc/node.1          | 3 +++
 src/env-inl.h       | 3 ++-
 src/node_options.cc | 5 +++++
 src/node_options.h  | 1 +
 5 files changed, 20 insertions(+), 1 deletion(-)

diff --git a/doc/api/cli.md b/doc/api/cli.md
index 7c0d5e88d4d640..f6e9e6cfcaf63a 100644
--- a/doc/api/cli.md
+++ b/doc/api/cli.md
@@ -619,6 +619,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
@@ -1431,6 +1439,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
index 9b285dedad1366..d127bb84cc7f3c 100644
--- a/doc/node.1
+++ b/doc/node.1
@@ -282,6 +282,9 @@ Disable the `node-addons` exports condition as well as disable loading native
 addons. When `--no-addons` is specified, calling `process.dlopen` or requiring
 a native C++ addon will fail and throw an exception.
 .
+.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-inl.h b/src/env-inl.h
index 7c3838dd71a625..2b000ed9ace5f7 100644
--- a/src/env-inl.h
+++ b/src/env-inl.h
@@ -887,7 +887,8 @@ inline bool Environment::hide_console_windows() const {
 }
 
 inline bool Environment::no_global_search_paths() const {
-  return flags_ & EnvironmentFlags::kNoGlobalSearchPaths;
+  return (flags_ & EnvironmentFlags::kNoGlobalSearchPaths) ||
+         !options_->global_search_paths;
 }
 
 bool Environment::filehandle_close_warning() const {
diff --git a/src/node_options.cc b/src/node_options.cc
index 519dd4ce01262d..c7f153f1d87817 100644
--- a/src/node_options.cc
+++ b/src/node_options.cc
@@ -407,6 +407,11 @@ EnvironmentOptionsParser::EnvironmentOptionsParser() {
             &EnvironmentOptions::allow_native_addons,
             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
index d3899c4c39ddf7..e43bb15a9a0c6e 100644
--- 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 allow_native_addons = true;
+  bool global_search_paths = true;
   bool warnings = true;
   bool force_context_aware = false;
   bool pending_deprecation = false;