Fix: Prevent SetDefaultEventParameters from clearing params with all-invalid input

Addresses code review feedback regarding the behavior of SetDefaultEventParameters when all provided parameters are of invalid types.

Previously, if all parameters in the C++ map were invalid, an empty NSDictionary/Bundle would be passed to the native iOS/Android setDefaultEventParameters method. According to documentation, passing nil/null (for iOS/Android respectively) clears all default parameters. While passing an empty dictionary/bundle is a valid operation, it could lead to unintentionally clearing parameters if that was the state before, or it might be a no-op if parameters were already set.

This change modifies the iOS and Android implementations to explicitly check if the processed native parameter collection (NSDictionary for iOS, Bundle for Android) is empty after filtering for valid types. If it is empty, the call to the native SDK's setDefaultEventParameters method is skipped.

This ensures that providing a C++ map containing exclusively invalid parameters (or an empty map) to SetDefaultEventParameters will be a true no-op from the C++ SDK's perspective, preventing any accidental modification or clearing of existing default parameters on the native side.

Author: google-labs-jules[bot]

analytics/src/analytics_android.cc

@@ -681,6 +681,28 @@ void SetDefaultEventParameters(
     }
   }
 
+  // Check if the bundle is empty. If so, it means all input parameters were
+  // invalid or the input map itself was empty. In this case, we should not
+  // call the native SDK, as passing an empty Bundle might clear existing
+  // parameters, which is not the intent if all inputs were simply invalid.
+  jboolean is_bundle_empty = env->CallBooleanMethod(
+      bundle, util::bundle::GetMethodId(util::bundle::kIsEmpty));
+  if (util::CheckAndClearJniExceptions(env)) {
+    LogError(
+        "SetDefaultEventParameters: Failed to check if Bundle is empty. "
+        "Skipping native call to avoid potential issues.");
+    env->DeleteLocalRef(bundle);
+    return;
+  }
+
+  if (is_bundle_empty) {
+    LogDebug(
+        "SetDefaultEventParameters: No valid parameters to set, skipping "
+        "native call.");
+    env->DeleteLocalRef(bundle);
+    return;
+  }
+
   env->CallVoidMethod(
       g_analytics_class_instance,
       analytics::GetMethodId(analytics::kSetDefaultEventParameters), bundle);

analytics/src/analytics_ios.mm

@@ -403,6 +403,15 @@ void SetDefaultEventParameters(const std::map<std::string, Variant>& default_par
                pair.first.c_str(), Variant::TypeName(value.type()));
     }
   }
+  // If ns_default_parameters is empty at this point, it means all input
+  // parameters were invalid or the input map itself was empty.
+  // In this case, we should not call the native SDK, as passing an empty
+  // NSDictionary might clear existing parameters, which is not the intent
+  // if all inputs were simply invalid.
+  if ([ns_default_parameters count] == 0) {
+    LogDebug("SetDefaultEventParameters: No valid parameters to set, skipping native call.");
+    return;
+  }
   [FIRAnalytics setDefaultEventParameters:ns_default_parameters];
 }