Add exception messages

Author: bording

projects/RabbitMQ.Client/Exceptions/PublishException.cs

@@ -53,6 +53,17 @@ public PublishException(ulong publishSequenceNumber, bool isReturn) : base()
             _publishSequenceNumber = publishSequenceNumber;
         }
 
+        public PublishException(ulong publishSequenceNumber, bool isReturn, string message) : base(message)
+        {
+            if (publishSequenceNumber == ulong.MinValue)
+            {
+                throw new ArgumentException($"{nameof(publishSequenceNumber)} must not be 0");
+            }
+
+            _isReturn = isReturn;
+            _publishSequenceNumber = publishSequenceNumber;
+        }
+
         /// <summary>
         /// <c>true</c> if this exception is due to a <c>basic.return</c>
         /// </summary>
@@ -76,10 +87,10 @@ public class PublishReturnException : PublishException
         private readonly ushort _replyCode;
         private readonly string _replyText;
 
-        public PublishReturnException(ulong publishSequenceNumber,
+        public PublishReturnException(ulong publishSequenceNumber, string message,
             string? exchange = null, string? routingKey = null,
             ushort? replyCode = null, string? replyText = null)
-            : base(publishSequenceNumber, true)
+            : base(publishSequenceNumber, true, message)
         {
             _exchange = exchange ?? string.Empty;
             _routingKey = routingKey ?? string.Empty;
@@ -116,11 +127,13 @@ internal static PublishException Create(bool isReturn,
         {
             if (isReturn)
             {
-                return new PublishReturnException(deliveryTag, exchange, routingKey, replyCode, replyText);
+                string message = $"{replyCode} {replyText} Exchange: {exchange} Routing Key: {routingKey}";
+                return new PublishReturnException(deliveryTag, message, exchange, routingKey, replyCode, replyText);
             }
             else
             {
-                return new PublishException(deliveryTag, isReturn);
+                string message = "Message rejected by broker.";
+                return new PublishException(deliveryTag, isReturn, message);
             }
         }
     }

projects/RabbitMQ.Client/PublicAPI.Unshipped.txt

@@ -1,6 +1,7 @@
+RabbitMQ.Client.Exceptions.PublishException.PublishException(ulong publishSequenceNumber, bool isReturn, string! message) -> void
 RabbitMQ.Client.Exceptions.PublishReturnException
 RabbitMQ.Client.Exceptions.PublishReturnException.Exchange.get -> string!
-RabbitMQ.Client.Exceptions.PublishReturnException.PublishReturnException(ulong publishSequenceNumber, string? exchange = null, string? routingKey = null, ushort? replyCode = null, string? replyText = null) -> void
+RabbitMQ.Client.Exceptions.PublishReturnException.PublishReturnException(ulong publishSequenceNumber, string! message, string? exchange = null, string? routingKey = null, ushort? replyCode = null, string? replyText = null) -> void
 RabbitMQ.Client.Exceptions.PublishReturnException.ReplyCode.get -> ushort
 RabbitMQ.Client.Exceptions.PublishReturnException.ReplyText.get -> string!
 RabbitMQ.Client.Exceptions.PublishReturnException.RoutingKey.get -> string!