Merge pull request #1 from datacamp-engineering/catch-errors-socket-w…

…rite Catch errors on socket write
datacamp-engineering · May 10, 2021 · 080e129 · 080e129
2 parents 2ea4ca4 + fd3d300
commit 080e129
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/lib/http-proxy/passes/ws-incoming.js b/lib/http-proxy/passes/ws-incoming.js
@@ -112,8 +112,12 @@ module.exports = {
     proxyReq.on('response', function (res) {
       // if upgrade event isn't going to happen, close the socket
       if (!res.upgrade && !socket.destroyed) {
-        socket.write(createHttpHeader('HTTP/' + res.httpVersion + ' ' + res.statusCode + ' ' + res.statusMessage, res.headers));
-        res.pipe(socket);
+        try {
+          socket.write(createHttpHeader('HTTP/' + res.httpVersion + ' ' + res.statusCode + ' ' + res.statusMessage, res.headers));
+          res.pipe(socket);
+        } catch (error) {
+          console.error(error)
+        }
       }
     });
 

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@datacamp/http-proxy",
-  "version": "1.18.2",
+  "version": "1.18.3",
   "repository": {
     "type": "git",
     "url": "https://github.com/datacamp-engineering/node-http-proxy.git"