Skip to content

Commit bcd8a56

Browse files
committed
close websocket if proxyReq is closed before upgrade
avoids leaving client sockets open when upstream connections are rejected.
1 parent 7730548 commit bcd8a56

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

lib/http-proxy/passes/ws-incoming.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,6 +89,10 @@ var passes = exports;
8989
);
9090
// Error Handler
9191
proxyReq.on('error', onOutgoingError);
92+
proxyReq.on('response', function (res) {
93+
// if upgrade event isn't going to happen, close the socket
94+
if (!res.upgrade) socket.end();
95+
});
9296

9397
proxyReq.on('upgrade', function(proxyRes, proxySocket, proxyHead) {
9498
proxySocket.on('error', onOutgoingError);

0 commit comments

Comments
 (0)