Fix audit forwarding when saga audit messages are enabled (#4904)

* fix null ref in Audit (#4894)

fixes #4893

* Specify list capacity to avoid resizing

* Use collection expression

---------

Co-authored-by: Simon Cropp <[email protected]>

Author: bording

src/ServiceControl.Audit/Auditing/AuditIngestor.cs

@@ -70,8 +70,7 @@ public async Task Ingest(List<MessageContext> contexts, CancellationToken cancel
 
         Task Forward(IReadOnlyCollection<MessageContext> messageContexts, string forwardingAddress, CancellationToken cancellationToken)
         {
-            var transportOperations = new TransportOperation[messageContexts.Count]; //We could allocate based on the actual number of ProcessedMessages but this should be OK
-            var index = 0;
+            var transportOperations = new List<TransportOperation>(messageContexts.Count);
             MessageContext anyContext = null;
             foreach (var messageContext in messageContexts)
             {
@@ -90,13 +89,12 @@ Task Forward(IReadOnlyCollection<MessageContext> messageContexts, string forward
                 // Forwarded messages should last as long as possible
                 outgoingMessage.Headers.Remove(Headers.TimeToBeReceived);
 
-                transportOperations[index] = new TransportOperation(outgoingMessage, new UnicastAddressTag(forwardingAddress));
-                index++;
+                transportOperations.Add(new TransportOperation(outgoingMessage, new UnicastAddressTag(forwardingAddress)));
             }
 
             return anyContext != null
                 ? messageDispatcher.Value.Dispatch(
-                    new TransportOperations(transportOperations),
+                    new TransportOperations([.. transportOperations]),
                     anyContext.TransportTransaction, cancellationToken)
                 : Task.CompletedTask;
         }