fix(node/http(s)): Set the url protocol by default

node/http.ts

@@ -90,6 +90,7 @@ export interface RequestOptions {
 
 /** ClientRequest represents the http(s) request from the client */
 class ClientRequest extends NodeWritable {
+  defaultProtocol = "http:";
   body: null | ReadableStream = null;
   controller: ReadableStreamDefaultController | null = null;
   constructor(
@@ -173,9 +174,9 @@ class ClientRequest extends NodeWritable {
         path,
         port,
       } = opts;
-      return `${protocol}//${auth ? `${auth}@` : ""}${host ?? hostname}${
-        port ? `:${port}` : ""
-      }${path}`;
+      return `${protocol ?? this.defaultProtocol}//${auth ? `${auth}@` : ""}${
+        host ?? hostname
+      }${port ? `:${port}` : ""}${path || ""}`;
     }
   }
 }

node/http_test.ts

@@ -116,3 +116,25 @@ Deno.test("[node/http chunked response", async () => {
     await promise;
   }
 });
+
+Deno.test("[node/http] request default protocol", async () => {
+  const promise = deferred<void>();
+  const server = http.createServer((_, res) => {
+    res.end("ok");
+  });
+  server.listen(() => {
+    const req = http.request(
+      { host: "localhost", port: server.address().port },
+      (res) => {
+        res.on("data", () => {});
+        res.on("end", () => {
+          server.close();
+          promise.resolve();
+        });
+        assertEquals(res.statusCode, 200);
+      },
+    );
+    req.end();
+  });
+  await promise;
+});

node/https.ts

@@ -57,6 +57,7 @@ export function get(...args: any[]) {
 export const globalAgent = undefined;
 /** HttpsClientRequest class loosely follows http.ClientRequest class API. */
 class HttpsClientRequest extends ClientRequest {
+  override defaultProtocol = "https:";
   override async _createCustomClient(): Promise<
     DenoUnstable.HttpClient | undefined
   > {