Merge pull request #560 from embrace-io/add-payload-flag

Add remote config that controls whether v2 payload is sent or not

embrace-android-sdk/src/main/java/io/embrace/android/embracesdk/config/behavior/SessionBehavior.kt

@@ -115,6 +115,11 @@ internal class SessionBehavior(
  fun shouldSendFullForErrorLog() =
  getFullSessionEvents().contains(SessionGatingKeys.FULL_SESSION_ERROR_LOGS)
 
+ /**
+ * Whether to use the V2 payload format for sending sessions.
+ */
+ fun useV2Payload() = remote?.sessionConfig?.useV2Payload ?: false
+
  /**
  * Checks whether a feature should be gated.
  * If [getSessionComponents] is null, this will return false.

embrace-android-sdk/src/main/java/io/embrace/android/embracesdk/config/remote/SessionRemoteConfig.kt

@@ -13,6 +13,9 @@ internal data class SessionRemoteConfig(
  @Json(name = "enable")
  val isEnabled: Boolean? = null,
 
+ @Json(name = "use_v2_payload")
+ val useV2Payload: Boolean? = null,
+
  /**
  * A list of session components (i.e. Breadcrumbs, Session properties, etc) that will be
  * included in the session payload. If components list exists, the services should restrict

embrace-android-sdk/src/test/java/io/embrace/android/embracesdk/EmbraceGatingServiceTest.kt

@@ -478,9 +478,9 @@ internal class EmbraceGatingServiceTest {
  private fun buildCustomRemoteConfig(components: Set<String>?, fullSessionEvents: Set<String>?) =
  RemoteConfig(
  sessionConfig = SessionRemoteConfig(
- true,
- components,
- fullSessionEvents
+ isEnabled = true,
+ sessionComponents = components,
+ fullSessionEvents = fullSessionEvents
  )
  )
 }

embrace-android-sdk/src/test/java/io/embrace/android/embracesdk/config/behavior/SessionBehaviorTest.kt

@@ -21,6 +21,7 @@ internal class SessionBehaviorTest {
  private val remote = RemoteConfig(
  sessionConfig = SessionRemoteConfig(
  isEnabled = true,
+ useV2Payload = true,
  sessionComponents = setOf("test"),
  fullSessionEvents = setOf("test2")
  ),
@@ -36,6 +37,7 @@ internal class SessionBehaviorTest {
  assertFalse(isGatingFeatureEnabled())
  assertFalse(isSessionControlEnabled())
  assertEquals(10, getMaxSessionProperties())
+ assertFalse(useV2Payload())
  }
  }
 
@@ -46,6 +48,7 @@ internal class SessionBehaviorTest {
  assertEquals(setOf("breadcrumbs"), getSessionComponents())
  assertEquals(setOf("crash"), getFullSessionEvents())
  assertTrue(isGatingFeatureEnabled())
+ assertFalse(useV2Payload())
  }
  }
 
@@ -57,6 +60,7 @@ internal class SessionBehaviorTest {
  assertEquals(setOf("test"), getSessionComponents())
  assertEquals(setOf("test2"), getFullSessionEvents())
  assertEquals(57, getMaxSessionProperties())
+ assertTrue(useV2Payload())
  }
  }