Sort rrweb events

romtsn · romtsn · commit 51cc43260dc5 · 2024-04-18T18:01:50.000+02:00
diff --git a/sentry-android-replay/src/main/java/io/sentry/android/replay/capture/BaseCaptureStrategy.kt b/sentry-android-replay/src/main/java/io/sentry/android/replay/capture/BaseCaptureStrategy.kt
@@ -220,7 +220,7 @@ internal abstract class BaseCaptureStrategy(
                             breadcrumbType = "default"
                             category = breadcrumbCategory
                             message = breadcrumbMessage
-                            data = if (breadcrumbData.isEmpty()) null else breadcrumbData
+                            data = breadcrumbData
                         }
                     }
                 }
@@ -229,7 +229,7 @@ internal abstract class BaseCaptureStrategy(
 
         val recording = ReplayRecording().apply {
             this.segmentId = segmentId
-            payload = recordingPayload
+            payload = recordingPayload.sortedBy { it.timestamp }
         }
 
         return ReplaySegment.Created(

Original file line number	Diff line number	Diff line change
`@@ -220,7 +220,7 @@ internal abstract class BaseCaptureStrategy(`
`220`	`220`	`breadcrumbType = "default"`
`221`	`221`	`category = breadcrumbCategory`
`222`	`222`	`message = breadcrumbMessage`
`223`		`- data = if (breadcrumbData.isEmpty()) null else breadcrumbData`
	`223`	`+ data = breadcrumbData`
`224`	`224`	`}`
`225`	`225`	`}`
`226`	`226`	`}`
`@@ -229,7 +229,7 @@ internal abstract class BaseCaptureStrategy(`
`229`	`229`
`230`	`230`	`val recording = ReplayRecording().apply {`
`231`	`231`	`this.segmentId = segmentId`
`232`		`- payload = recordingPayload`
	`232`	`+ payload = recordingPayload.sortedBy { it.timestamp }`
`233`	`233`	`}`
`234`	`234`
`235`	`235`	`return ReplaySegment.Created(`