Merge pull request #2652 from DataDog/mconstantin/fix-error-log-message

dd-mergequeue[bot] · mariusc83 · web-flow · commit da39148f0816 · 2025-05-15T14:14:12.000Z
Fix logging format for error messages in PlainBatchFileReaderWriter

Co-authored-by: mariusc83 &lt;marius.constantin@datadoghq.com&gt;
diff --git a/dd-sdk-android-core/src/main/kotlin/com/datadog/android/core/internal/persistence/file/batch/PlainBatchFileReaderWriter.kt b/dd-sdk-android-core/src/main/kotlin/com/datadog/android/core/internal/persistence/file/batch/PlainBatchFileReaderWriter.kt
@@ -78,7 +78,7 @@ internal class PlainBatchFileReaderWriter(
             internalLogger.log(
                 InternalLogger.Level.ERROR,
                 listOf(InternalLogger.Target.MAINTAINER, InternalLogger.Target.TELEMETRY),
-                { "ERROR_READ.format(Locale.US, file.path)" },
+                { ERROR_READ.format(Locale.US, file.path) },
                 e
             )
             emptyList()

Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,7 @@ internal class PlainBatchFileReaderWriter(`
`78`	`78`	`internalLogger.log(`
`79`	`79`	`InternalLogger.Level.ERROR,`
`80`	`80`	`listOf(InternalLogger.Target.MAINTAINER, InternalLogger.Target.TELEMETRY),`
`81`		`- { "ERROR_READ.format(Locale.US, file.path)" },`
	`81`	`+ { ERROR_READ.format(Locale.US, file.path) },`
`82`	`82`	`e`
`83`	`83`	`)`
`84`	`84`	`emptyList()`