Merge pull request #1275 from atsign-foundation/gkc-urgent-20230321

fix: Fix bug introduced in 3.0.28
atsign-foundation · Mar 21, 2023 · 5472db9 · 5472db9
2 parents 4727174 + 48d4b57
commit 5472db9
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 13 deletions.
diff --git a/packages/at_secondary_server/lib/src/connection/outbound/outbound_client_manager.dart b/packages/at_secondary_server/lib/src/connection/outbound/outbound_client_manager.dart
@@ -60,11 +60,6 @@ class OutboundClientManager {
     return newClient;
   }
 
-  close() {
-    closed = true;
-    _pool.close();
-  }
-
   int getActiveConnectionSize() {
     return _pool.getActiveConnectionSize();
   }

diff --git a/packages/at_secondary_server/lib/src/connection/outbound/outbound_client_pool.dart b/packages/at_secondary_server/lib/src/connection/outbound/outbound_client_pool.dart
@@ -19,11 +19,6 @@ class OutboundClientPool {
     return _clients.length < size;
   }
 
-  close() {
-    closed = true;
-    clearAllClients();
-  }
-
   /// Removes the least recently used OutboundClient from the pool. Returns the removed client,
   /// or returns null if there are fewer than 2 items currently in the pool.
   OutboundClient? removeLeastRecentlyUsed() {

diff --git a/packages/at_secondary_server/lib/src/server/at_secondary_impl.dart b/packages/at_secondary_server/lib/src/server/at_secondary_impl.dart
@@ -550,9 +550,6 @@ class AtSecondaryServerImpl implements AtSecondaryServer {
       logger.info("Terminating all inbound connections");
       inboundConnectionFactory.removeAllConnections();
 
-      logger.info("Terminating all outbound connections");
-      outboundClientManager.close();
-
       logger.info("Stopping Notification Resource Manager");
       notificationResourceManager.stop();