fix(ext/node): fix whatwg url formatting

[kt3k]

This PR fixes the bug of url.format of node:url module (which affects the execution of remix)

Currently the following script prints http:https://localhost:3000:3000/:

import url from "node:url";
console.log(url.format(new URL("http:https://localhost:3000")));

This PR fixes the above bug and it now prints http:https://localhost:3000/. (This PR also implements unicode option handling of url.format)

[bartlomieju]

@kt3k any chance this could be related to bartlomieju/vite-deno-example#7?

[kt3k]

In the case of remix, the stack trace look like the below:

Remix App Server started at http:https://localhost:3000 (http:https://10.2.10.173:3000)
TypeError: Invalid URL: 'http:https://localhost:3000:3000/'
    at getSerialization (ext:deno_url/00_url.js:89:11)
    at new URL (ext:deno_url/00_url.js:359:27)
    at requestHandler (file:https:///Users/kt3k/tmp/y/node_modules/.deno/@[email protected]/node_modules/@remix-run/server-runtime/dist/server.js:32:15)
    at file:https:///Users/kt3k/tmp/y/node_modules/.deno/@[email protected]/node_modules/@remix-run/express/dist/server.js:39:28
    at eventLoopTick (ext:core/01_core.js:181:11)
Terminating Deno.serve loop due to unexpected error TypeError: Cannot read properties of undefined (reading 'status')
    at ext:deno_http/00_serve.js:544:26
    at eventLoopTick (ext:core/01_core.js:188:13)

The actual invalid url (http:https://localhost:3000:3000/) is passed to URL constructor. So I guess it might be slightly different issue.

[bartlomieju]

LGTM