Add fix for custom sampling rules

Author: andrewlock

tracer/src/Datadog.Trace/Configuration/MutableSettings.cs

@@ -679,8 +679,24 @@ public static MutableSettings CreateUpdatedMutableSettings(
         var dynamicConfig = new ConfigurationBuilder(dynamicSource, telemetry);
         var manualCustomSamplingRules = manualConfig.WithKeys(ConfigurationKeys.CustomSamplingRules).AsStringResult();
         var remoteCustomSamplingRules = dynamicConfig.WithKeys(ConfigurationKeys.CustomSamplingRules).GetAsClassResult<string>(validator: null, converter: s => s);
-        var customSamplingRules = GetCombinedResult(manualCustomSamplingRules, remoteCustomSamplingRules, initialSettings.CustomSamplingRules);
-        var customSamplingRulesIsRemote = remoteCustomSamplingRules.ConfigurationResult.IsValid;
+        string? customSamplingRules;
+        bool customSamplingRulesIsRemote;
+
+        if (remoteCustomSamplingRules.ConfigurationResult is { IsValid: true, Result: var r2 })
+        {
+            customSamplingRules = r2;
+            customSamplingRulesIsRemote = false;
+        }
+        else if (manualCustomSamplingRules.ConfigurationResult is { IsValid: true, Result: var r1 })
+        {
+            customSamplingRules = r1;
+            customSamplingRulesIsRemote = true;
+        }
+        else
+        {
+            customSamplingRules = initialSettings.CustomSamplingRules;
+            customSamplingRulesIsRemote = initialSettings.CustomSamplingRulesIsRemote;
+        }
 
         // These can't actually be changed in code right now, so just set them to the same values
         var gitRepositoryUrl = initialSettings.GitRepositoryUrl;
@@ -714,24 +730,6 @@ public static MutableSettings CreateUpdatedMutableSettings(
             errorLog: errorLog,
             telemetry: telemetry);
 
-        static string? GetCombinedResult(
-            ConfigurationBuilder.ClassConfigurationResultWithKey<string> manualResult,
-            ConfigurationBuilder.ClassConfigurationResultWithKey<string> remoteResult,
-            string? fallback)
-        {
-            if (manualResult.ConfigurationResult is { IsValid: true, Result: var r1 })
-            {
-                return r1;
-            }
-
-            if (remoteResult.ConfigurationResult is { IsValid: true, Result: var r2 })
-            {
-                return r2;
-            }
-
-            return fallback;
-        }
-
         static ReadOnlyDictionary<string, string> GetHeaderTagsResult(
             ConfigurationBuilder.ClassConfigurationResultWithKey<IDictionary<string, string>> result,
             bool headerTagsNormalizationFixEnabled,