fixed: Sockets hanging (closes #7097) (#2976)

* fixed: Sockets hanging

* added past solution

* refactor

* bump version

Author: PavelMor25

package-lock.json

@@ -1,7 +1,7 @@
 {
   "name": "testcafe-hammerhead",
   "description": "A powerful web-proxy used as a core for the TestCafe testing framework (https://github.com/DevExpress/testcafe).",
-  "version": "31.6.3",
+  "version": "31.6.4",
   "homepage": "https://github.com/DevExpress/testcafe-hammerhead",
   "bugs": {
     "url": "https://github.com/DevExpress/testcafe-hammerhead/issues"

package.json

@@ -426,6 +426,15 @@ export default class RequestPipelineContext extends BaseRequestPipelineContext {
         this.goToNextStage = false;
     }
 
+    closeConnectionOnError (): void {
+        this.req.destroy();
+
+        // NOTE: For Node versions greater than 15.4, we check the socket. If req.destroy didn't destroy it, we call the corresponding method of the socket.
+        if (this.req.socket && !this.req.socket.destroyed)
+            this.req.socket.destroy();
+
+    }
+
     toProxyUrl (url: string, isCrossDomain: boolean, resourceType: string, charset?: string, reqOrigin?: string, credentials?: urlUtils.Credentials): string {
         const proxyHostname = this.serverInfo.hostname;
         const proxyProtocol = this.serverInfo.protocol;

src/request-pipeline/context/index.ts

@@ -90,7 +90,7 @@ export function error (ctx: RequestPipelineContext, err: string) {
     if (ctx.isPage && !ctx.isIframe)
         ctx.session.handlePageError(ctx, err);
     else if (ctx.isAjax)
-        ctx.req.destroy();
+        ctx.closeConnectionOnError();
     else
         ctx.closeWithError(500, err.toString());
 }