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

…ction limit is reached (#1515)
libp2p · Dec 16, 2022 · 24f7709 · 24f7709
1 parent 4058909
commit 24f7709
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/src/connection-manager/index.ts b/src/connection-manager/index.ts
@@ -690,7 +690,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