Merge pull request #3394 from DataDog/release/3.9.0

Merge `3.9.1` into `master`

Author: kikoveiga

CHANGELOG.md

@@ -1,3 +1,8 @@
+# 3.9.1 / 2026-04-23
+
+* [BUGFIX] Cache reflection lookups in `LayoutNodeUtils`. See [#3381](https://github.com/DataDog/dd-sdk-android/pull/3381)
+* [IMPROVEMENT] Add `fed2` endpoint to `DatadogSite`. See [#3391](https://github.com/DataDog/dd-sdk-android/pull/3391)
+
 # 3.9.0 / 2026-04-16
 
 * [FEATURE] Track GraphQL errors. See [#3275](https://github.com/DataDog/dd-sdk-android/pull/3275)

buildSrc/src/main/kotlin/com/datadog/gradle/config/AndroidConfig.kt

@@ -19,7 +19,7 @@ object AndroidConfig {
     const val MIN_SDK_FOR_AUTO = 29
     const val BUILD_TOOLS_VERSION = "36.0.0"
 
-    val VERSION = Version(3, 9, 0, Version.Type.Release)
+    val VERSION = Version(3, 9, 1, Version.Type.Release)
 }
 
 // TODO RUM-628 Switch to Java 17 bytecode

dd-sdk-android-core/api/apiSurface

@@ -23,6 +23,7 @@ enum com.datadog.android.DatadogSite
   - AP1
   - AP2
   - US1_FED
+  - US2_FED
   - STAGING
   val intakeEndpoint: String
 class com.datadog.android._InternalProxy

dd-sdk-android-core/api/dd-sdk-android-core.api

@@ -64,6 +64,7 @@ public final class com/datadog/android/DatadogSite : java/lang/Enum {
 	public static final field STAGING Lcom/datadog/android/DatadogSite;
 	public static final field US1 Lcom/datadog/android/DatadogSite;
 	public static final field US1_FED Lcom/datadog/android/DatadogSite;
+	public static final field US2_FED Lcom/datadog/android/DatadogSite;
 	public static final field US3 Lcom/datadog/android/DatadogSite;
 	public static final field US5 Lcom/datadog/android/DatadogSite;
 	public final fun getIntakeEndpoint ()Ljava/lang/String;

dd-sdk-android-core/src/main/kotlin/com/datadog/android/DatadogSite.kt

@@ -50,6 +50,11 @@ enum class DatadogSite private constructor(internal val siteName: String, privat
      */
     US1_FED("us1_fed", "browser-intake-ddog-gov.com"),
 
+    /**
+     *  The US2_FED site (FedRAMP compatible): [us2.ddog-gov.com](https://us2.ddog-gov.com).
+     */
+    US2_FED("us2_fed", "browser-intake-us2-ddog-gov.com"),
+
     /**
      *  The STAGING site (internal usage only): [app.datad0g.com](https://app.datad0g.com).
      */

dd-sdk-android-core/src/test/kotlin/com/datadog/android/DatadogSiteTest.kt

@@ -45,6 +45,11 @@ internal class DatadogSiteTest {
         assertThat(DatadogSite.US1_FED.intakeEndpoint).isEqualTo("https://browser-intake-ddog-gov.com")
     }
 
+    @Test
+    fun `M return intake endpoint W intakeEndpoint {US2-FED}`() {
+        assertThat(DatadogSite.US2_FED.intakeEndpoint).isEqualTo("https://browser-intake-us2-ddog-gov.com")
+    }
+
     @Test
     fun `M return intake endpoint W intakeEndpoint {EU1}`() {
         assertThat(DatadogSite.EU1.intakeEndpoint).isEqualTo("https://browser-intake-datadoghq.eu")

detekt_custom_safe_calls.yml

@@ -954,16 +954,19 @@ datadog:
       - "kotlin.collections.MutableMap.filterKeys(kotlin.Function1)"
       - "kotlin.collections.MutableMap.filterValues(kotlin.Function1)"
       - "kotlin.collections.MutableMap.forEach(kotlin.Function1)"
+      - "kotlin.collections.MutableMap.get(java.lang.Class)"
       - "kotlin.collections.MutableMap.get(kotlin.String)"
       - "kotlin.collections.MutableMap.get(kotlin.String?)"
       - "kotlin.collections.MutableMap.getOrElse(java.lang.Class, kotlin.Function0)"
+      - "kotlin.collections.MutableMap.getOrPut(java.lang.Class, kotlin.Function0)"
       - "kotlin.collections.MutableMap.getOrPut(kotlin.String, kotlin.Function0)"
       - "kotlin.collections.MutableMap.isEmpty()"
       - "kotlin.collections.MutableMap.isNotEmpty()"
       - "kotlin.collections.MutableMap.iterator()"
       - "kotlin.collections.MutableMap.orEmpty()"
       - "kotlin.collections.MutableMap.map(kotlin.Function1)"
       - "kotlin.collections.MutableMap.mapValues(kotlin.Function1)"
+      - "kotlin.collections.MutableMap.put(java.lang.Class, kotlin.String?)"
       - "kotlin.collections.MutableMap.put(kotlin.Any?, kotlin.Any?)"
       - "kotlin.collections.MutableMap.put(kotlin.String, kotlin.Any)"
       - "kotlin.collections.MutableMap.put(kotlin.String, kotlin.Any?)"

features/dd-sdk-android-flags/src/main/kotlin/com/datadog/android/flags/internal/DatadogSiteExtensions.kt

@@ -25,7 +25,8 @@ internal fun DatadogSite.getFlagsEndpoint(customerDomain: String): String? {
  * @return Flags host string in format `<customerDomain>.ff-cdn.<site>.<tld>`, or null if site not supported
  */
 private fun DatadogSite.flagsHost(customerDomain: String): String? = when (this) {
-    DatadogSite.US1_FED -> null
+    DatadogSite.US1_FED,
+    DatadogSite.US2_FED -> null
 
     DatadogSite.STAGING -> "$customerDomain.ff-cdn.datad0g.com"
     DatadogSite.EU1 -> buildFlagsHostString(customerDomain, tld = "eu") // No site in the host, .eu TLD

features/dd-sdk-android-flags/src/test/kotlin/com/datadog/android/flags/internal/DatadogSiteExtensionsTest.kt

@@ -10,10 +10,10 @@ import com.datadog.android.DatadogSite
 import fr.xgouchet.elmyr.annotation.StringForgery
 import fr.xgouchet.elmyr.junit5.ForgeExtension
 import org.assertj.core.api.Assertions.assertThat
-import org.junit.jupiter.api.Test
 import org.junit.jupiter.api.extension.ExtendWith
 import org.junit.jupiter.params.ParameterizedTest
 import org.junit.jupiter.params.provider.Arguments
+import org.junit.jupiter.params.provider.EnumSource
 import org.junit.jupiter.params.provider.MethodSource
 
 @ExtendWith(ForgeExtension::class)
@@ -56,10 +56,14 @@ internal class DatadogSiteExtensionsTest {
 
     // region getFlagsEndpoint - Error Cases
 
-    @Test
-    fun `M return null W getFlagsEndpoint() { unsupported site }`(@StringForgery customerDomain: String) {
+    @ParameterizedTest
+    @EnumSource(DatadogSite::class, names = ["US1_FED", "US2_FED"])
+    fun `M return null W getFlagsEndpoint() { unsupported site }`(
+        site: DatadogSite,
+        @StringForgery customerDomain: String
+    ) {
         // When
-        val result = DatadogSite.US1_FED.getFlagsEndpoint(customerDomain)
+        val result = site.getFlagsEndpoint(customerDomain)
 
         // Then
         assertThat(result).isNull()

features/dd-sdk-android-flags/src/test/kotlin/com/datadog/android/flags/internal/net/PrecomputedAssignmentsRequestFactoryTest.kt

@@ -24,6 +24,8 @@ import org.json.JSONObject
 import org.junit.jupiter.api.BeforeEach
 import org.junit.jupiter.api.Test
 import org.junit.jupiter.api.extension.ExtendWith
+import org.junit.jupiter.params.ParameterizedTest
+import org.junit.jupiter.params.provider.EnumSource
 import org.mockito.Mock
 import org.mockito.junit.jupiter.MockitoExtension
 import org.mockito.junit.jupiter.MockitoSettings
@@ -49,7 +51,10 @@ internal class PrecomputedAssignmentsRequestFactoryTest {
     @BeforeEach
     fun `set up`(forge: Forge) {
         fakeDatadogContext = fakeDatadogContext.copy(
-            site = forge.aValueFrom(DatadogSite::class.java, exclude = listOf(DatadogSite.US1_FED)),
+            site = forge.aValueFrom(
+                DatadogSite::class.java,
+                exclude = listOf(DatadogSite.US1_FED, DatadogSite.US2_FED)
+            ),
             featuresContext = fakeDatadogContext.featuresContext +
                 mapOf(Feature.RUM_FEATURE_NAME to mapOf("application_id" to fakeRumApplicationId.toString()))
         )
@@ -252,8 +257,10 @@ internal class PrecomputedAssignmentsRequestFactoryTest {
 
     // region create() - Error cases
 
-    @Test
+    @ParameterizedTest
+    @EnumSource(DatadogSite::class, names = ["US1_FED", "US2_FED"])
     fun `M return null W create() { unsupported site and no custom endpoint }`(
+        site: DatadogSite,
         @StringForgery fakeTargetingKey: String
     ) {
         // Given
@@ -262,7 +269,7 @@ internal class PrecomputedAssignmentsRequestFactoryTest {
             attributes = emptyMap()
         )
         fakeDatadogContext = fakeDatadogContext.copy(
-            site = DatadogSite.US1_FED
+            site = site
         )
 
         // When