Fix navigation span end

It looks like we weren't ending the span when a new navigation span was being started.
grafana · Sep 12, 2024 · c61a234 · c61a234
1 parent 4f43248
commit c61a234
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/common/frame_session.go b/common/frame_session.go
@@ -797,6 +797,11 @@ func (fs *FrameSession) processNavigationSpan(url string, id cdp.FrameID) {
 		return
 	}
 
+	// End the navigation span if it is non-nil
+	if fs.mainFrameSpan != nil {
+		fs.mainFrameSpan.End()
+	}
+
 	_, fs.mainFrameSpan = TraceNavigation(
 		fs.ctx, fs.targetID.String(), trace.WithAttributes(attribute.String("navigation.url", url)),
 	)