Fix Json Formatting for AggregateExceptions (#57)

serilog-contrib · Sep 21, 2021 · a381464 · a381464
1 parent 0e2d41a
commit a381464
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/Serilog.Sinks.Grafana.Loki/LokiJsonTextFormatter.cs b/src/Serilog.Sinks.Grafana.Loki/LokiJsonTextFormatter.cs
@@ -165,7 +165,7 @@ protected virtual void SerializeException(TextWriter output, Exception exception
                 var count = aggregateException.InnerExceptions.Count;
                 for (var i = 0; i < count; i++)
                 {
-                    var isLast = i == count;
+                    var isLast = i == count - 1;
                     SerializeException(output, aggregateException.InnerExceptions[i], level + 1);
                     if (!isLast)
                     {

diff --git a/...log.Sinks.Grafana.Loki.Tests/IntegrationTests/LokiJsonTextFormatterRequestPayloadTests.cs b/...log.Sinks.Grafana.Loki.Tests/IntegrationTests/LokiJsonTextFormatterRequestPayloadTests.cs
@@ -180,7 +180,7 @@ public void AggregateExceptionShouldBeSerializedCorrectly()
                         .Replace(s, "\"[0-9]{19}\"", "\"<unixepochinnanoseconds>\"");
                     return Regex.Replace(
                         s,
-                        @"(?<=\\u0022StackTrace)(.*?)(?=}})",
+                        @"(?<=\\u0022StackTrace)(.*?)}\](?=}})",
                         @"<stack-trace>");
                 });
             });