Merge pull request #2528 from DataDog/nogorodnikov/remove-double-body-computation-in-rum-upload

0xnm · web-flow · commit f37baf177821 · 2025-03-10T09:39:38.000+01:00
Remove double computation of the RUM payload
diff --git a/features/dd-sdk-android-rum/src/main/kotlin/com/datadog/android/rum/internal/net/RumRequestFactory.kt b/features/dd-sdk-android-rum/src/main/kotlin/com/datadog/android/rum/internal/net/RumRequestFactory.kt
@@ -51,12 +51,7 @@ internal class RumRequestFactory(
                 idempotencyKey,
                 context
             ),
-            body = viewEventFilter.filterOutRedundantViewEvents(batchData)
-                .map { it.data }
-                .join(
-                    separator = PAYLOAD_SEPARATOR,
-                    internalLogger = internalLogger
-                ),
+            body = body,
             contentType = RequestFactory.CONTENT_TYPE_TEXT_UTF8
         )
     }

Original file line number	Diff line number	Diff line change
`@@ -51,12 +51,7 @@ internal class RumRequestFactory(`
`51`	`51`	`idempotencyKey,`
`52`	`52`	`context`
`53`	`53`	`),`
`54`		`- body = viewEventFilter.filterOutRedundantViewEvents(batchData)`
`55`		`- .map { it.data }`
`56`		`- .join(`
`57`		`- separator = PAYLOAD_SEPARATOR,`
`58`		`- internalLogger = internalLogger`
`59`		`- ),`
	`54`	`+ body = body,`
`60`	`55`	`contentType = RequestFactory.CONTENT_TYPE_TEXT_UTF8`
`61`	`56`	`)`
`62`	`57`	`}`