Serialize sequence number as long or string.

Fixes issue raised in #1805

Encode publish sequence number in x-dotnet-pub-seq-no as a `long` instead of ASCII string, unless value is greater than `long.MaxValue`.

Author: lukebakken

projects/Applications/PublisherConfirms/PublisherConfirms.cs

@@ -30,13 +30,13 @@
 //---------------------------------------------------------------------------
 
 using System;
-using System.Buffers.Binary;
 using System.Collections.Generic;
 using System.Diagnostics;
 using System.Text;
 using System.Threading;
 using System.Threading.Tasks;
 using RabbitMQ.Client;
+using RabbitMQ.Client.Exceptions;
 
 const ushort MAX_OUTSTANDING_CONFIRMS = 256;
 
@@ -54,7 +54,7 @@
     Persistent = true
 };
 
-string hostname = "localhost";
+string hostname = "shostakovich";
 if (args.Length > 0)
 {
     if (false == string.IsNullOrWhiteSpace(args[0]))
@@ -124,11 +124,43 @@ async Task PublishMessagesInBatchAsync()
     var sw = new Stopwatch();
     sw.Start();
 
+    channel.BasicReturnAsync += (sender, ea) =>
+    {
+        string sequenceNumber = string.Empty;
+
+        IReadOnlyBasicProperties props = ea.BasicProperties;
+        if (props.Headers is not null)
+        {
+            object? maybeSeqNum = props.Headers[Constants.PublishSequenceNumberHeader];
+            if (maybeSeqNum is not null)
+            {
+                switch (maybeSeqNum)
+                {
+                    case byte[] seqNumBytes:
+                        sequenceNumber = Encoding.ASCII.GetString(seqNumBytes);
+                        break;
+                    case string seqNumStr:
+                        sequenceNumber = seqNumStr;
+                        break;
+                }
+            }
+        }
+
+        Console.WriteLine($"{DateTime.Now} [WARNING] message sequence number {sequenceNumber} has been basic.return-ed");
+
+        return Task.CompletedTask;
+    };
+
     var publishTasks = new List<ValueTask>();
     for (int i = 0; i < MESSAGE_COUNT; i++)
     {
+        string rk = queueName;
+        if (i % 1000 == 0)
+        {
+            rk = Guid.NewGuid().ToString();
+        }
         byte[] body = Encoding.UTF8.GetBytes(i.ToString());
-        publishTasks.Add(channel.BasicPublishAsync(exchange: string.Empty, routingKey: queueName, body: body, mandatory: true, basicProperties: props));
+        publishTasks.Add(channel.BasicPublishAsync(exchange: string.Empty, routingKey: rk, body: body, mandatory: true, basicProperties: props));
         outstandingMessageCount++;
 
         if (outstandingMessageCount == batchSize)
@@ -139,9 +171,13 @@ async Task PublishMessagesInBatchAsync()
                 {
                     await pt;
                 }
+                catch (PublishException pex)
+                {
+                    Console.Error.WriteLine($"{DateTime.Now} [ERROR] saw nack or return, pex.IsReturn: '{pex.IsReturn}', seq no: '{pex.PublishSequenceNumber}'");
+                }
                 catch (Exception ex)
                 {
-                    Console.Error.WriteLine($"{DateTime.Now} [ERROR] saw nack or return, ex: '{ex}'");
+                    Console.Error.WriteLine($"{DateTime.Now} [ERROR] saw exception, ex: '{ex}'");
                 }
             }
             publishTasks.Clear();
@@ -157,9 +193,13 @@ async Task PublishMessagesInBatchAsync()
             {
                 await pt;
             }
+            catch (PublishException pex)
+            {
+                Console.Error.WriteLine($"{DateTime.Now} [ERROR] saw nack or return, pex.IsReturn: '{pex.IsReturn}', seq no: '{pex.PublishSequenceNumber}'");
+            }
             catch (Exception ex)
             {
-                Console.Error.WriteLine($"{DateTime.Now} [ERROR] saw nack or return, ex: '{ex}'");
+                Console.Error.WriteLine($"{DateTime.Now} [ERROR] saw exception, ex: '{ex}'");
             }
         }
         publishTasks.Clear();
@@ -244,13 +284,14 @@ async Task CleanOutstandingConfirms(ulong deliveryTag, bool multiple)
         if (props.Headers is not null)
         {
             object? maybeSeqNum = props.Headers[Constants.PublishSequenceNumberHeader];
-            if (maybeSeqNum is not null)
+            if (maybeSeqNum is long longSequenceNumber)
             {
-                sequenceNumber = BinaryPrimitives.ReadUInt64BigEndian((byte[])maybeSeqNum);
+                sequenceNumber = (ulong)longSequenceNumber;
             }
         }
 
-        Console.WriteLine($"{DateTime.Now} [WARNING] message sequence number {sequenceNumber} has been basic.return-ed");
+        Console.WriteLine($"{DateTime.Now} [INFO] message sequence number '{sequenceNumber}' has been basic.return-ed");
+
         return CleanOutstandingConfirms(sequenceNumber, false);
     };
 
@@ -290,13 +331,21 @@ async Task CleanOutstandingConfirms(ulong deliveryTag, bool multiple)
             // This will cause a basic.return, for fun
             rk = Guid.NewGuid().ToString();
         }
+
+        var msgProps = new BasicProperties
+        {
+            Persistent = true,
+            Headers = new Dictionary<string, object?>()
+        };
+
+        msgProps.Headers.Add(Constants.PublishSequenceNumberHeader, (long)nextPublishSeqNo);
+
         (ulong, ValueTask) data =
-            (nextPublishSeqNo, channel.BasicPublishAsync(exchange: string.Empty, routingKey: rk, body: body, mandatory: true, basicProperties: props));
+            (nextPublishSeqNo, channel.BasicPublishAsync(exchange: string.Empty, routingKey: rk, body: body, mandatory: true, basicProperties: msgProps));
         publishTasks.Add(data);
     }
 
     using var cts = new CancellationTokenSource(TimeSpan.FromSeconds(10));
-    // await Task.WhenAll(publishTasks).WaitAsync(cts.Token);
     foreach ((ulong SeqNo, ValueTask PublishTask) datum in publishTasks)
     {
         try

projects/RabbitMQ.Client/Impl/Channel.BasicPublish.cs

@@ -36,7 +36,6 @@
 using System.Threading;
 using System.Threading.Tasks;
 using RabbitMQ.Client.Framing;
-using RabbitMQ.Client.Util;
 
 namespace RabbitMQ.Client.Impl
 {
@@ -217,9 +216,14 @@ void MaybeAddPublishSequenceNumberToHeaders(IDictionary<string, object?> headers
             {
                 if (_publisherConfirmationsEnabled && _publisherConfirmationTrackingEnabled)
                 {
-                    byte[] publishSequenceNumberBytes = new byte[8];
-                    NetworkOrderSerializer.WriteUInt64(ref publishSequenceNumberBytes.GetStart(), publishSequenceNumber);
-                    headers[Constants.PublishSequenceNumberHeader] = publishSequenceNumberBytes;
+                    if (publishSequenceNumber > long.MaxValue)
+                    {
+                        headers[Constants.PublishSequenceNumberHeader] = publishSequenceNumber.ToString();
+                    }
+                    else
+                    {
+                        headers[Constants.PublishSequenceNumberHeader] = (long)publishSequenceNumber;
+                    }
                 }
             }
         }

projects/RabbitMQ.Client/Impl/Channel.PublisherConfirms.cs

@@ -34,13 +34,13 @@
 using System.Collections.Generic;
 using System.Diagnostics;
 using System.Runtime.CompilerServices;
+using System.Text;
 using System.Threading;
 using System.Threading.RateLimiting;
 using System.Threading.Tasks;
 using RabbitMQ.Client.Events;
 using RabbitMQ.Client.Exceptions;
 using RabbitMQ.Client.Framing;
-using RabbitMQ.Client.Util;
 
 namespace RabbitMQ.Client.Impl
 {
@@ -233,10 +233,20 @@ private void HandleReturn(BasicReturnEventArgs basicReturnEvent)
                 ulong publishSequenceNumber = 0;
                 IReadOnlyBasicProperties props = basicReturnEvent.BasicProperties;
                 object? maybeSeqNum = props.Headers?[Constants.PublishSequenceNumberHeader];
-                if (maybeSeqNum != null &&
-                    maybeSeqNum is byte[] seqNumBytes)
+                if (maybeSeqNum != null)
                 {
-                    publishSequenceNumber = NetworkOrderDeserializer.ReadUInt64(seqNumBytes);
+                    switch (maybeSeqNum)
+                    {
+                        case long seqNumLong:
+                            publishSequenceNumber = (ulong)seqNumLong;
+                            break;
+                        case string seqNumString:
+                            publishSequenceNumber = ulong.Parse(seqNumString);
+                            break;
+                        case byte[] seqNumBytes:
+                            publishSequenceNumber = ulong.Parse(Encoding.ASCII.GetString(seqNumBytes));
+                            break;
+                    }
                 }
 
                 HandleNack(publishSequenceNumber, multiple: false, isReturn: true);