fix: always try to remap both span and error stacktraces (#12221) (#1…

…2331) When an event contains both a span and an error, the error is ignored by the sourcemap processor leading to stacktraces not being remapped. (cherry picked from commit 8f10084) Co-authored-by: kruskall <[email protected]>
elastic · Jan 3, 2024 · 8222e01 · 8222e01
1 parent db730dd
commit 8222e01
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/internal/sourcemap/processor.go b/internal/sourcemap/processor.go
@@ -57,10 +57,10 @@ func (p BatchProcessor) ProcessBatch(ctx context.Context, batch *modelpb.Batch)
 		if event.GetService().GetName() == "" || event.GetService().GetVersion() == "" {
 			continue
 		}
-		switch {
-		case event.Span != nil:
+		if event.Span != nil {
 			p.processStacktraceFrames(ctx, event.Service, event.Span.Stacktrace...)
-		case event.Error != nil:
+		}
+		if event.Error != nil {
 			if event.Error.Log != nil {
 				p.processStacktraceFrames(ctx, event.Service, event.Error.Log.Stacktrace...)
 			}