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

When an event contains both a span and an error, the error is ignored by the sourcemap processor leading to stacktraces not being remapped.
elastic · Dec 15, 2023 · 8f10084 · 8f10084
1 parent d636ad3
commit 8f10084
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...)
 			}