[GenevaExporter] Add ability to export Log Exception using StackTrace

src/OpenTelemetry.Exporter.Geneva/.publicApi/PublicAPI.Unshipped.txt

@@ -0,0 +1 @@
+OpenTelemetry.Exporter.Geneva.ExceptionStackExportMode.ExportAsStackTraceString = 2 -> OpenTelemetry.Exporter.Geneva.ExceptionStackExportMode

src/OpenTelemetry.Exporter.Geneva/CHANGELOG.md

@@ -2,6 +2,9 @@
 
 ## Unreleased
 
+* Add support for exporting exception stack using `Exception.StackTrace`.
+  This can be enabled via enum `ExceptionStackExportMode.ExportAsStackTraceString`.
+
 ## 1.10.0
 
 Released 2024-Nov-18

src/OpenTelemetry.Exporter.Geneva/ExceptionStackExportMode.cs

@@ -14,9 +14,17 @@ public enum ExceptionStackExportMode
     Drop,
 
     /// <summary>
-    /// Exception stack traces are exported as strings.
+    /// Exception stack traces are exported as string, using a culture agnostic representation of the results of ToString() implementation of the Exception.
+    /// Though formatted in culture-agnostic way, this is more targetted towards human readability.
+    /// See <see href="https://learn.microsoft.com/dotnet/api/system.exception.stacktrace"/>.
     /// </summary>
     ExportAsString,
 
+    /// <summary>
+    /// Exception stack traces are exported as string, using the StackTrace property of the Exception.
+    /// See <see href="https://learn.microsoft.com/dotnet/api/system.exception.tostring"/>.
+    /// </summary>
+    ExportAsStackTraceString,
+
     // ExportAsArrayOfStacks - future if stacks can be exported in more structured way
 }

src/OpenTelemetry.Exporter.Geneva/Internal/MsgPack/MsgPackLogExporter.cs

@@ -417,22 +417,32 @@ internal ArraySegment<byte> SerializeLogRecord(LogRecord logRecord)
 
             cursor = MessagePackSerializer.SerializeAsciiString(buffer, cursor, "env_ex_msg");
             cursor = MessagePackSerializer.SerializeUnicodeString(buffer, cursor, logRecord.Exception.Message);
-            cntFields += 1;
-
+            cntFields += 1;
+
+            // The current approach relies on the existing trim
+            // capabilities which trims string in excess of STRING_SIZE_LIMIT_CHAR_COUNT
+            // TODO: Revisit this:
+            // 1. Trim it off based on how much more bytes are available
+            // before running out of limit instead of STRING_SIZE_LIMIT_CHAR_COUNT.
+            // 2. Trim smarter, by trimming the middle of stack, an
+            // keep top and bottom.
             if (this.exportExceptionStack == ExceptionStackExportMode.ExportAsString)
             {
-                // The current approach relies on the existing trim
-                // capabilities which trims string in excess of STRING_SIZE_LIMIT_CHAR_COUNT
-                // TODO: Revisit this:
-                // 1. Trim it off based on how much more bytes are available
-                // before running out of limit instead of STRING_SIZE_LIMIT_CHAR_COUNT.
-                // 2. Trim smarter, by trimming the middle of stack, an
-                // keep top and bottom.
                 var exceptionStack = logRecord.Exception.ToInvariantString();
                 cursor = MessagePackSerializer.SerializeAsciiString(buffer, cursor, "env_ex_stack");
                 cursor = MessagePackSerializer.SerializeUnicodeString(buffer, cursor, exceptionStack);
                 cntFields += 1;
             }
+            else if (this.exportExceptionStack == ExceptionStackExportMode.ExportAsStackTraceString)
+            {
+                var exceptionStack = logRecord.Exception.StackTrace;
+                if (exceptionStack != null)
+                {
+                    cursor = MessagePackSerializer.SerializeAsciiString(buffer, cursor, "env_ex_stack");
+                    cursor = MessagePackSerializer.SerializeUnicodeString(buffer, cursor, exceptionStack);
+                    cntFields += 1;
+                }
+            }
         }
 
         MessagePackSerializer.WriteUInt16(buffer, idxMapSizePatch, cntFields);

src/OpenTelemetry.Exporter.Geneva/Internal/Tld/TldLogExporter.cs

@@ -260,6 +260,15 @@ internal EventBuilder SerializeLogRecord(LogRecord logRecord)
                 eb.AddCountedAnsiString("ext_ex_stack", exceptionStack, Encoding.UTF8, 0, Math.Min(exceptionStack.Length, StringLengthLimit));
                 partAFieldsCount++;
             }
+            else if (this.exceptionStackExportMode == ExceptionStackExportMode.ExportAsStackTraceString)
+            {
+                var stackTrace = logRecord.Exception.StackTrace;
+                if (!string.IsNullOrEmpty(stackTrace))
+                {
+                    eb.AddCountedAnsiString("ext_ex_stack", stackTrace, Encoding.UTF8, 0, Math.Min(stackTrace.Length, StringLengthLimit));
+                    partAFieldsCount++;
+                }
+            }
         }
 
         eb.SetStructFieldCount(partAFieldsCountPatch, partAFieldsCount);