lib: stop installing fetch if no_browser_globals is true

RaisinTen · web-flow · commit a82c1a1da710 · 2022-02-16T16:53:43.000Z
Fixes: #41816 Signed-off-by: Darshan Sen <raisinten@gmail.com> PR-URL: #41969 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Nitzan Uziely <linkgoron@gmail.com> Reviewed-By: Filip Skokan <panva.ip@gmail.com> Reviewed-By: Mestery <mestery@protonmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
diff --git a/lib/internal/bootstrap/pre_execution.js b/lib/internal/bootstrap/pre_execution.js
@@ -151,7 +151,8 @@ function setupWarningHandler() {
 
 // https://fetch.spec.whatwg.org/
 function setupFetch() {
-  if (!getOptionValue('--experimental-fetch')) {
+  if (process.config.variables.node_no_browser_globals ||
+      !getOptionValue('--experimental-fetch')) {
     return;
   }
 

Original file line number	Diff line number	Diff line change
`@@ -151,7 +151,8 @@ function setupWarningHandler() {`
`151`	`151`
`152`	`152`	`// https://fetch.spec.whatwg.org/`
`153`	`153`	`function setupFetch() {`
`154`		`- if (!getOptionValue('--experimental-fetch')) {`
	`154`	`+ if (process.config.variables.node_no_browser_globals \|\|`
	`155`	`+ !getOptionValue('--experimental-fetch')) {`
`155`	`156`	`return;`
`156`	`157`	`}`
`157`	`158`