fix: should not prune a peer that is in the allow list once the conne…

…ction limit is reached (libp2p#1515)
maschad · Jan 7, 2023 · 80dc00d · 80dc00d
1 parent d8c8e8c
commit 80dc00d
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/src/connection-manager/index.ts b/src/connection-manager/index.ts
@@ -683,7 +683,15 @@ export class DefaultConnectionManager extends EventEmitter<ConnectionManagerEven
 
     for (const connection of sortedConnections) {
       log('too many connections open - closing a connection to %p', connection.remotePeer)
-      toClose.push(connection)
+      // check allow list
+      const connectionInAllowList = this.allow.some((ma) => {
+        return ma.getPeerId() === connection.remotePeer.toString()
+      })
+
+      // Connections in the allow list should be excluded from pruning
+      if (!connectionInAllowList) {
+        toClose.push(connection)
+      }
 
       if (toClose.length === toPrune) {
         break