Converted optional parameter in RabbitMQTransport constructor to an o…

…verload where parameter is required
Particular · Dec 18, 2024 · 482e061 · 482e061
1 parent 524ab57
commit 482e061
Showing 1 changed file with 24 additions and 3 deletions.
diff --git a/src/NServiceBus.Transport.RabbitMQ/RabbitMQTransport.cs b/src/NServiceBus.Transport.RabbitMQ/RabbitMQTransport.cs
@@ -25,13 +25,23 @@ public class RabbitMQTransport : TransportDefinition
 
         readonly List<(string hostName, int port, bool useTls)> additionalClusterNodes = [];
 
+        /// <summary>
+        /// Creates a new instance of the RabbitMQ transport.
+        /// </summary>
+        /// <param name="routingTopology">The routing topology to use.</param>
+        /// <param name="connectionString">The connection string to use when connecting to the broker.</param>
+        public RabbitMQTransport(RoutingTopology routingTopology, string connectionString)
+            : this(routingTopology, connectionString, null)
+        {
+        }
+
         /// <summary>
         /// Creates a new instance of the RabbitMQ transport.
         /// </summary>
         /// <param name="routingTopology">The routing topology to use.</param>
         /// <param name="connectionString">The connection string to use when connecting to the broker.</param>
         /// <param name="managementConnectionString">The connection string to use when connecting to the management API</param>
-        public RabbitMQTransport(RoutingTopology routingTopology, string connectionString, string managementConnectionString = null)
+        public RabbitMQTransport(RoutingTopology routingTopology, string connectionString, string managementConnectionString)
             : base(TransportTransactionMode.ReceiveOnly,
                 supportsDelayedDelivery: true,
                 supportsPublishSubscribe: true,
@@ -53,9 +63,20 @@ public RabbitMQTransport(RoutingTopology routingTopology, string connectionStrin
         /// </summary>
         /// <param name="routingTopology">The routing topology to use.</param>
         /// <param name="connectionString">The connection string to use when connecting to the broker.</param>
-        /// <param name="managementConnectionString">The connection string to use when connecting to the management API</param>
         /// <param name="enableDelayedDelivery">Should the delayed delivery infrastructure be created by the endpoint</param>
-        public RabbitMQTransport(RoutingTopology routingTopology, string connectionString, bool enableDelayedDelivery, string managementConnectionString = null)
+        public RabbitMQTransport(RoutingTopology routingTopology, string connectionString, bool enableDelayedDelivery)
+            : this(routingTopology, connectionString, enableDelayedDelivery, null)
+        {
+        }
+
+        /// <summary>
+        /// Creates a new instance of the RabbitMQ transport.
+        /// </summary>
+        /// <param name="routingTopology">The routing topology to use.</param>
+        /// <param name="connectionString">The connection string to use when connecting to the broker.</param>
+        /// <param name="enableDelayedDelivery">Should the delayed delivery infrastructure be created by the endpoint</param>
+        /// <param name="managementConnectionString">The connection string to use when connecting to the management API</param>
+        public RabbitMQTransport(RoutingTopology routingTopology, string connectionString, bool enableDelayedDelivery, string managementConnectionString)
             : base(TransportTransactionMode.ReceiveOnly,
                 supportsDelayedDelivery: enableDelayedDelivery,
                 supportsPublishSubscribe: true,