Merge pull request #9008 from abpframework/liangshiwei/patch-1

Remove item if connection creation fails

framework/src/Volo.Abp.RabbitMQ/Volo/Abp/RabbitMQ/ConnectionPool.cs

@@ -25,16 +25,26 @@ namespace Volo.Abp.RabbitMQ
         {
             connectionName ??= RabbitMqConnections.DefaultConnectionName;
 
-            return Connections.GetOrAdd(
+            try
-                connectionName, () => new Lazy<IConnection>(() =>
+            {
-                {
+                var lazyConnection = Connections.GetOrAdd(
-                    var connection = Options.Connections.GetOrDefault(connectionName);
+                    connectionName, () => new Lazy<IConnection>(() =>
-                    var hostnames = connection.HostName.TrimEnd(';').Split(';');
+                    {
-                    // Handle Rabbit MQ Cluster.
+                        var connection = Options.Connections.GetOrDefault(connectionName);
-                    return hostnames.Length == 1 ? connection.CreateConnection() : connection.CreateConnection(hostnames);
+                        var hostnames = connection.HostName.TrimEnd(';').Split(';');
+                        // Handle Rabbit MQ Cluster.
+                        return hostnames.Length == 1 ? connection.CreateConnection() : connection.CreateConnection(hostnames);
+
+                    })
+                );
 
-                })
+                return lazyConnection.Value;
-            ).Value;
+            }
+            catch (Exception)
+            {
+                Connections.TryRemove(connectionName, out _);
+                throw;
+            }
         }
 
         public void Dispose()