Merge pull request #2376 from DataDog/mconstantin/rum-6400/add-log-fo…

…r-sdk-initialization-outside-main-process RUM-6400 Add warning log when initializing the SDK outside of main process
DataDog · Nov 7, 2024 · a7c4f31 · a7c4f31
2 parents ab89e64 + 97c83d1
commit a7c4f31
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 0 deletions.
diff --git a/dd-sdk-android-core/src/main/kotlin/com/datadog/android/core/internal/CoreFeature.kt b/dd-sdk-android-core/src/main/kotlin/com/datadog/android/core/internal/CoreFeature.kt
@@ -626,6 +626,13 @@ internal class CoreFeature(
         } else {
             appContext.packageName == currentProcess.processName
         }
+        if (!isMainProcess) {
+            internalLogger.log(
+                InternalLogger.Level.WARN,
+                InternalLogger.Target.USER,
+                { SDK_INITIALIZED_IN_SECONDARY_PROCESS_WARNING_MESSAGE }
+            )
+        }
     }
 
     private fun shutDownExecutors() {
@@ -676,6 +683,10 @@ internal class CoreFeature(
     // endregion
 
     companion object {
+        internal const val SDK_INITIALIZED_IN_SECONDARY_PROCESS_WARNING_MESSAGE =
+            "Datadog SDK was initialized in a secondary process: although data will still be captured," +
+                " nothing will be uploaded from this process. Make sure to also initialize the SDK from the main" +
+                " process of your application."
 
         internal val DEFAULT_FLUSHABLE_EXECUTOR_SERVICE_FACTORY =
             FlushableExecutorService.Factory { logger, executorContext, backPressureStrategy ->

diff --git a/dd-sdk-android-core/src/test/kotlin/com/datadog/android/core/internal/CoreFeatureTest.kt b/dd-sdk-android-core/src/test/kotlin/com/datadog/android/core/internal/CoreFeatureTest.kt
@@ -781,6 +781,11 @@ internal class CoreFeatureTest {
 
         // Then
         assertThat(testedFeature.isMainProcess).isFalse()
+        mockInternalLogger.verifyLog(
+            InternalLogger.Level.WARN,
+            InternalLogger.Target.USER,
+            CoreFeature.SDK_INITIALIZED_IN_SECONDARY_PROCESS_WARNING_MESSAGE
+        )
     }
 
     @Test