Merge pull request #46 from Mastercard/fix/sonar-fix

karen-avetisyan-mc · web-flow · commit c0bb4d4f66cd · 2025-07-25T12:55:18.000+01:00
Fix: sonar issue: csharpsquid:S6444. Denial of Service security hotspot
diff --git a/Mastercard.Developer.ClientEncryption.Core/Utils/JsonUtils.cs b/Mastercard.Developer.ClientEncryption.Core/Utils/JsonUtils.cs
@@ -7,7 +7,7 @@ namespace Mastercard.Developer.ClientEncryption.Core.Utils
 {
     internal static class JsonUtils
     {
-        private static readonly Regex LastElementInPathRegExp = new Regex(".*(\\['.*'\\])"); // Returns "['obj2']" for "$['obj1']['obj2']"
+        private static readonly Regex LastElementInPathRegExp = new Regex(".*(\\['.*'\\])", RegexOptions.Compiled, TimeSpan.FromMilliseconds(100)); // Returns "['obj2']" for "$['obj1']['obj2']"
         
         /// <summary>
         /// Get JSON path to the parent of the object at the given JSON path.

Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@ namespace Mastercard.Developer.ClientEncryption.Core.Utils`
`7`	`7`	`{`
`8`	`8`	`internal static class JsonUtils`
`9`	`9`	`{`
`10`		`- private static readonly Regex LastElementInPathRegExp = new Regex(".(\\['.'\\])"); // Returns "['obj2']" for "$['obj1']['obj2']"`
	`10`	`+ private static readonly Regex LastElementInPathRegExp = new Regex(".(\\['.'\\])", RegexOptions.Compiled, TimeSpan.FromMilliseconds(100)); // Returns "['obj2']" for "$['obj1']['obj2']"`
`11`	`11`
`12`	`12`	`/// <summary>`
`13`	`13`	`/// Get JSON path to the parent of the object at the given JSON path.`