esm: identify parent importing a url with invalid host

PR-URL: #49736 Backport-PR-URL: #50669 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Geoffrey Booth <[email protected]>

Author: JakobJingleheimer

lib/internal/modules/esm/resolve.js

@@ -255,7 +255,16 @@ function finalizeResolution(resolved, base, preserveSymlinks){
 resolved.pathname,'must not include encoded "/" or "\\" characters',
 fileURLToPath(base));
 
-letpath=fileURLToPath(resolved);
+letpath;
+try{
+path=fileURLToPath(resolved);
+}catch(err){
+const{ setOwnProperty }=require('internal/util');
+setOwnProperty(err,'input',`${resolved}`);
+setOwnProperty(err,'module',`${base}`);
+throwerr;
+}
+
 if(getOptionValue('--experimental-specifier-resolution')==='node'){
 letfile=resolveExtensionsWithTryExactName(resolved);
 

test/es-module/test-esm-loader-default-resolver.mjs

@@ -49,4 +49,18 @@ describe('default resolver', () =>{
 assert.strictEqual(stdout.trim(),'index.byoe!');
 assert.strictEqual(stderr,'');
 });
+
+it('should identify the parent module of an invalid URL host in import specifier',async()=>{
+if(process.platform==='win32')return;
+
+const{ code, stderr }=awaitspawnPromisified(execPath,[
+'--no-warnings',
+fixtures.path('es-modules','invalid-posix-host.mjs'),
+]);
+
+assert.match(stderr,/ERR_INVALID_FILE_URL_HOST/);
+assert.match(stderr,/file:\/\/hmm\.js/);
+assert.match(stderr,/invalid-posix-host\.mjs/);
+assert.strictEqual(code,1);
+});
 });

test/fixtures/es-modules/invalid-posix-host.mjs

@@ -0,0 +1 @@
+import"file://hmm.js";