fix(node-http-handler): clear obsolete socket timeout listeners (#1531)

* fix(node-http-handler): clear obsolete socket timeout listeners

* Add changeset

Author: tomekt-bolt

.changeset/twenty-peaches-notice.md

@@ -0,0 +1,5 @@
+---
+"@smithy/node-http-handler": patch
+---
+
+Clear obsolete timeout handlers from socket.

packages/node-http-handler/src/set-socket-timeout.spec.ts

@@ -49,6 +49,7 @@ describe("setSocketTimeout", () => {
       socket: {
         setTimeout: vi.fn(),
       },
+      on: vi.fn(),
     };
 
     it("calls setTimeout on the socket if it is available after deferral", async () => {
@@ -64,6 +65,8 @@ describe("setSocketTimeout", () => {
         expectedDeferredSocketTimeout,
         expect.any(Function)
       );
+      expect(clientRequestWithSocket.on).toHaveBeenCalledTimes(1);
+      expect(clientRequestWithSocket.on).toHaveBeenLastCalledWith("close", expect.any(Function));
     });
   });
 

packages/node-http-handler/src/set-socket-timeout.ts

@@ -19,6 +19,7 @@ export const setSocketTimeout = (
 
     if (request.socket) {
       request.socket.setTimeout(timeout, onTimeout);
+      request.on("close", () => request.socket?.removeListener("timeout", onTimeout));
     } else {
       request.setTimeout(timeout, onTimeout);
     }