Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit e89e0b4

Browse filesBrowse files
Trottdanielleadams
authored andcommitted
esm: remove superfluous argument
$defaultResolve() takes two arguments. The third argument is unused. PR-URL: #43884 Reviewed-By: Jacob Smith <jacob@frende.me> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Tobias Nießen <tniessen@tnie.de>
1 parent bb6787c commit e89e0b4
Copy full SHA for e89e0b4

File tree

Expand file treeCollapse file tree

1 file changed

+1
-1
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

1 file changed

+1
-1
lines changed
Open diff view settings
Collapse file

‎lib/internal/modules/esm/resolve.js‎

Copy file name to clipboardExpand all lines: lib/internal/modules/esm/resolve.js
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1230,7 +1230,7 @@ if (policy) {
12301230
specifier,
12311231
context
12321232
) {
1233-
const ret = await $defaultResolve(specifier, context, $defaultResolve);
1233+
const ret = await $defaultResolve(specifier, context);
12341234
// This is a preflight check to avoid data exfiltration by query params etc.
12351235
policy.manifest.mightAllow(ret.url, () =>
12361236
new ERR_MANIFEST_DEPENDENCY_MISSING(

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.