From e89e0b470b110ca6363ccf620e7b601aff69bb43 Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Mon, 18 Jul 2022 06:06:55 -0700 Subject: [PATCH] esm: remove superfluous argument MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit $defaultResolve() takes two arguments. The third argument is unused. PR-URL: https://github.com/nodejs/node/pull/43884 Reviewed-By: Jacob Smith Reviewed-By: Antoine du Hamel Reviewed-By: Ruben Bridgewater Reviewed-By: Tobias Nießen --- lib/internal/modules/esm/resolve.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/internal/modules/esm/resolve.js b/lib/internal/modules/esm/resolve.js index c6abfc9b2b3ecd..435a6f279e9fd1 100644 --- a/lib/internal/modules/esm/resolve.js +++ b/lib/internal/modules/esm/resolve.js @@ -1230,7 +1230,7 @@ if (policy) { specifier, context ) { - const ret = await $defaultResolve(specifier, context, $defaultResolve); + const ret = await $defaultResolve(specifier, context); // This is a preflight check to avoid data exfiltration by query params etc. policy.manifest.mightAllow(ret.url, () => new ERR_MANIFEST_DEPENDENCY_MISSING(