Fix closeTunnel() function (#128)

apify · Apr 15, 2021 · d34c9e0 · d34c9e0
1 parent 43e8c9d
commit d34c9e0
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,7 @@
+1.0.2 / 2021-04-14
+==================
+- Bugfix: `closeTunnel()` function didn't work because of `runningServers[port].connections.forEach is not a function` error (see issue [#127](https://github.com/apify/proxy-chain/issues/127))
+
 1.0.1 / 2021-04-09
 ==================
  - Bugfix: `parseUrl()` result now always includes port for `http(s)`, `ftp` and `ws(s)` (even if explicitly specified port is the default one)
@@ -34,7 +38,7 @@
 
 0.4.9 / 2021-01-26
 ===================
-- Bugfix: Added back the `scheme` field to result from`parseUrl()`
+- Bugfix: Added back the `scheme` field to result from `parseUrl()`
 
 0.4.8 / 2021-01-26
 ===================

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "proxy-chain",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "Node.js implementation of a proxy server (think Squid) with support for SSL, authentication, upstream proxy chaining, and protocol tunneling.",
   "main": "build/index.js",
   "keywords": [

diff --git a/src/tcp_tunnel_tools.js b/src/tcp_tunnel_tools.js
@@ -36,7 +36,7 @@ export function createTunnel(proxyUrl, targetHost, providedOptions = {}, callbac
     server.on('connection', (srcSocket) => {
         const port = server.address().port;
 
-        runningServers[port].connections = srcSocket;
+        runningServers[port].connections.push(srcSocket);
         const remoteAddress = `${srcSocket.remoteAddress}:${srcSocket.remotePort}`;
         log('new client connection from %s', remoteAddress);