fix(mongos): disconnect proxies which are not mongos instances

NODE-2232
mongodb · Oct 9, 2019 · ee53983 · ee53983
1 parent 37dbe49
commit ee53983
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/lib/core/topologies/mongos.js b/lib/core/topologies/mongos.js
@@ -366,7 +366,8 @@ function handleInitialConnectEvent(self, event) {
           self.s.logger.warn(f(message, _this.name));
         }
 
-        // This is not a mongos proxy, remove it completely
+        // This is not a mongos proxy, destroy and remove it completely
+        _this.destroy(true);
         removeProxyFrom(self.connectingProxies, _this);
         // Emit the left event
         self.emit('left', 'server', _this);
@@ -445,10 +446,9 @@ function connectProxies(self, servers) {
       server.connect(self.s.connectOptions);
     }, timeoutInterval);
   }
+
   // Start all the servers
-  while (servers.length > 0) {
-    connect(servers.shift(), timeoutInterval++);
-  }
+  servers.forEach(server => connect(server, timeoutInterval++));
 }
 
 function pickProxy(self, session) {