module: fix extension lookups for top-level main

guybedford · gibfahn · commit 415fb5673535 · 2017-10-31T00:15:02.000Z
The reason is that absolute URLs do not go through extension and index checks. By switching to an absolute path, the resolver still applies extensions properly to the top-level main. PR-URL: #16526 Fixes: #16476 Reviewed-By: Myles Borins <myles.borins@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
diff --git a/lib/module.js b/lib/module.js
@@ -452,7 +452,7 @@ Module._load = function(request, parent, isMain) {
           ESMLoader.hook(hooks);
         }
       }
-      await ESMLoader.import(getURLFromFilePath(request).href);
+      await ESMLoader.import(getURLFromFilePath(request).pathname);
     })()
     .catch((e) => {
       console.error(e);
diff --git a/test/parallel/test-module-main-extension-lookup.js b/test/parallel/test-module-main-extension-lookup.js
@@ -0,0 +1,7 @@
+'use strict';
+require('../common');
+const { execFileSync } = require('child_process');
+
+const node = process.argv[0];
+
+execFileSync(node, ['--experimental-modules', 'test/es-module/test-esm-ok']);

Original file line number	Diff line number	Diff line change
`@@ -452,7 +452,7 @@ Module._load = function(request, parent, isMain) {`
`452`	`452`	`ESMLoader.hook(hooks);`
`453`	`453`	`}`
`454`	`454`	`}`
`455`		`- await ESMLoader.import(getURLFromFilePath(request).href);`
	`455`	`+ await ESMLoader.import(getURLFromFilePath(request).pathname);`
`456`	`456`	`})()`
`457`	`457`	`.catch((e) => {`
`458`	`458`	`console.error(e);`