Allow leading/trailing whitespace in length prefix for (NOT) STARTS/E…

…NDS WITH ANY OF comparators to align behavior with other SDKs
configcat · Feb 22, 2024 · a17a9d0 · a17a9d0
1 parent 3bda929
commit a17a9d0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/ConfigCat.Client.Tests/ConfigV2EvaluationTests.cs b/src/ConfigCat.Client.Tests/ConfigV2EvaluationTests.cs
@@ -736,10 +736,10 @@ public void ComparisonAttributeTrimming_Test(string key, string expectedReturnVa
     [DataRow("notendswithanyof", "no trim")]
     [DataRow("arraycontainsanyof", "no trim")]
     [DataRow("arraynotcontainsanyof", "no trim")]
-    [DataRow("startwithanyofhashed", "default")]
-    [DataRow("notstartwithanyofhashed", "default")]
-    [DataRow("endswithanyofhashed", "default")]
-    [DataRow("notendswithanyofhashed", "default")]
+    [DataRow("startwithanyofhashed", "no trim")]
+    [DataRow("notstartwithanyofhashed", "no trim")]
+    [DataRow("endswithanyofhashed", "no trim")]
+    [DataRow("notendswithanyofhashed", "no trim")]
     //semver comparator values trimmed because of backward compatibility
     [DataRow("semverisoneof", "4 trim")]
     [DataRow("semverisnotoneof", "5 trim")]

diff --git a/src/ConfigCatClient/Evaluation/RolloutEvaluator.cs b/src/ConfigCatClient/Evaluation/RolloutEvaluator.cs
@@ -549,7 +549,7 @@ private static bool EvaluateSensitiveTextSliceEqualsAnyOf(string text, string[]?
 
             var index = item.IndexOf('_');
             if (index < 0
-                || !int.TryParse(item.AsSpan(0, index).ToParsable(), NumberStyles.None, CultureInfo.InvariantCulture, out var sliceLength)
+                || !int.TryParse(item.AsSpan(0, index).ToParsable(), NumberStyles.AllowLeadingWhite | NumberStyles.AllowTrailingWhite, CultureInfo.InvariantCulture, out var sliceLength)
                 || (hash2 = item.AsSpan(index + 1)).IsEmpty)
             {
                 EnsureComparisonValue<string>(null);