Issue fix (Azure#51573)

Author: HarveyLink

sdk/network/Azure.ResourceManager.Network/CHANGELOG.md

@@ -8,6 +8,8 @@
 
 ### Bugs Fixed
 
+- Fixed an issue in `ManagedRuleSetRuleGroup` deserialization where rule IDs could be either strings or numbers in JSON, causing `InvalidOperationException` when parsing mixed-type arrays.
+
 ### Other Changes
 
 ## 1.11.1 (2025-07-01)

sdk/network/Azure.ResourceManager.Network/src/Customization/Models/ManagedRuleSetRuleGroup.cs

@@ -0,0 +1,31 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
+#nullable disable
+
+using System.Collections.Generic;
+using System.Linq;
+using System.Runtime.CompilerServices;
+using System.Text.Json;
+using Azure.Core;
+
+namespace Azure.ResourceManager.Network.Models
+{
+    // Custom deserialization hook for handling mixed-type rule arrays.
+    // Addresses issue where WAF rule IDs can be either strings or numbers in JSON.
+    [CodeGenSerialization(nameof(Rules), DeserializationValueHook = nameof(DeserializeNumberValue))]
+    public partial class ManagedRuleSetRuleGroup
+    {
+        [MethodImpl(MethodImplOptions.AggressiveInlining)]
+        private static void DeserializeNumberValue(JsonProperty property, ref IReadOnlyList<string> rules)
+        {
+            if (property.Value.ValueKind == JsonValueKind.Null)
+                return;
+
+            rules = [.. property.Value.EnumerateArray()
+                .Select(item => item.ValueKind == JsonValueKind.String
+                    ? item.GetString()
+                    : item.GetInt32().ToString())];
+        }
+    }
+}

sdk/network/Azure.ResourceManager.Network/src/Generated/Models/ManagedRuleSetRuleGroup.Serialization.cs

@@ -11,6 +11,9 @@
     <None Update="TestData\DeploymentTemplate.json">
       <CopyToOutputDirectory>Never</CopyToOutputDirectory>
     </None>
+    <None Update="TestData\ManagedRuleSetRuleGroup.json">
+      <CopyToOutputDirectory>Always</CopyToOutputDirectory>
+    </None>
     <None Update="TestData\VmssDeploymentTemplate.json">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </None>

sdk/network/Azure.ResourceManager.Network/tests/Azure.ResourceManager.Network.Tests.csproj

@@ -0,0 +1,12 @@
+{
+  "ruleGroupName": "OWASP_3.2",
+  "description": "OWASP Core Rule Set 3.2",
+  "rules": [
+    "920100",
+    920110,
+    "920120",
+    920130,
+    "920140",
+    920150
+  ]
+}

sdk/network/Azure.ResourceManager.Network/tests/TestData/ManagedRuleSetRuleGroup.json

@@ -19,5 +19,25 @@ public void DeserializeChangeNumber()
             var data = AzureFirewallIPGroups.DeserializeAzureFirewallIPGroups(jsonContent.RootElement);
             Assert.NotNull(data.ChangeNumber);
         }
+
+        // Regression test for ManagedRuleSetRuleGroup deserialization with mixed string/number rule IDs
+        [Test]
+        public void DeserializeManagedRuleSetRuleGroupWithMixedRuleTypes()
+        {
+            using var sr = new StreamReader(Path.Combine("TestData", "ManagedRuleSetRuleGroup.json"));
+            using var jsonContent = JsonDocument.Parse(sr.BaseStream);
+            var data = ManagedRuleSetRuleGroup.DeserializeManagedRuleSetRuleGroup(jsonContent.RootElement);
+
+            Assert.NotNull(data.Rules);
+            Assert.AreEqual(6, data.Rules.Count);
+
+            // Verify that both string and numeric rule IDs are properly converted to strings
+            Assert.AreEqual("920100", data.Rules[0]); // Originally string
+            Assert.AreEqual("920110", data.Rules[1]); // Originally number
+            Assert.AreEqual("920120", data.Rules[2]); // Originally string
+            Assert.AreEqual("920130", data.Rules[3]); // Originally number
+            Assert.AreEqual("920140", data.Rules[4]); // Originally string
+            Assert.AreEqual("920150", data.Rules[5]); // Originally number
+        }
     }
 }