fix(node/http): don't leak resources on destroyed request (#20040)

Closes https://github.com/denoland/deno/issues/19828

1 files changed, 34 insertions, 0 deletions

diff --git a/cli/tests/unit_node/http_test.ts b/cli/tests/unit_node/http_test.ts
index 9feee0272..825815ae6 100644
--- a/cli/tests/unit_node/http_test.ts
+++ b/cli/tests/unit_node/http_test.ts
@@ -777,3 +777,37 @@ Deno.test("[node/http] server emits error if addr in use", async () => {
     `Wrong error: ${err.message}`,
   );
 });
+
+Deno.test(
+  "[node/http] client destroy doesn't leak",
+  { permissions: { net: true } },
+  async () => {
+    const ac = new AbortController();
+    let timerId;
+
+    const server = Deno.serve(
+      { port: 5929, signal: ac.signal },
+      async (_req) => {
+        await new Promise((resolve) => {
+          timerId = setTimeout(resolve, 5000);
+        });
+        return new Response("hello");
+      },
+    );
+    const promise = deferred();
+
+    const request = http.request("http://localhost:5929/");
+    request.on("error", promise.reject);
+    request.on("close", () => {});
+    request.end();
+    setTimeout(() => {
+      request.destroy(new Error());
+      promise.resolve();
+    }, 100);
+
+    await promise;
+    clearTimeout(timerId);
+    ac.abort();
+    await server.finished;
+  },
+);