Sample code correction in migrate-to-policy.md

isjwuk · web-flow · commit 440eae315065 · 2021-12-09T08:17:31.000Z
Corrected code sample so that the correct combinations of source address, source IP Group,Destination Address, Destination IP Group are applied in the if statements.
diff --git a/articles/firewall-manager/migrate-to-policy.md b/articles/firewall-manager/migrate-to-policy.md
@@ -121,13 +121,13 @@ If ($azfw.NetworkRuleCollections.Count -gt 0) {
                     }
                     elseif($rule.DestinationIpGroups)
                     {
-                        $firewallPolicyNetRule = New-AzFirewallPolicyNetworkRule -Name $rule.Name -SourceIpGroup $rule.SourceIpGroups -DestinationAddress $rule.DestinationAddresses -DestinationPort $rule.DestinationPorts -Protocol $rule.Protocols
+                        $firewallPolicyNetRule = New-AzFirewallPolicyNetworkRule -Name $rule.Name -SourceAddress $rule.SourceAddresses -DestinationIpGroup $rule.DestinationIpGroups -DestinationPort $rule.DestinationPorts -Protocol $rule.Protocols
                     }
                 }
                 elseif($rule.SourceIpGroups){
                     If($rule.DestinationAddresses)
                     {
-                        $firewallPolicyNetRule = New-AzFirewallPolicyNetworkRule -Name $rule.Name -SourceAddress $rule.SourceAddresses -DestinationIpGroup $rule.DestinationIpGroups -DestinationPort $rule.DestinationPorts -Protocol $rule.Protocols
+                        $firewallPolicyNetRule = New-AzFirewallPolicyNetworkRule -Name $rule.Name -SourceIpGroup $rule.SourceIpGroups -DestinationAddress $rule.DestinationAddresses -DestinationPort $rule.DestinationPorts -Protocol $rule.Protocols
                     }
                     elseif($rule.DestinationIpGroups)
                     {

Original file line number	Diff line number	Diff line change
`@@ -121,13 +121,13 @@ If ($azfw.NetworkRuleCollections.Count -gt 0) {`
`121`	`121`	`}`
`122`	`122`	`elseif($rule.DestinationIpGroups)`
`123`	`123`	`{`
`124`		`- $firewallPolicyNetRule = New-AzFirewallPolicyNetworkRule -Name $rule.Name -SourceIpGroup $rule.SourceIpGroups -DestinationAddress $rule.DestinationAddresses -DestinationPort $rule.DestinationPorts -Protocol $rule.Protocols`
	`124`	`+ $firewallPolicyNetRule = New-AzFirewallPolicyNetworkRule -Name $rule.Name -SourceAddress $rule.SourceAddresses -DestinationIpGroup $rule.DestinationIpGroups -DestinationPort $rule.DestinationPorts -Protocol $rule.Protocols`
`125`	`125`	`}`
`126`	`126`	`}`
`127`	`127`	`elseif($rule.SourceIpGroups){`
`128`	`128`	`If($rule.DestinationAddresses)`
`129`	`129`	`{`
`130`		`- $firewallPolicyNetRule = New-AzFirewallPolicyNetworkRule -Name $rule.Name -SourceAddress $rule.SourceAddresses -DestinationIpGroup $rule.DestinationIpGroups -DestinationPort $rule.DestinationPorts -Protocol $rule.Protocols`
	`130`	`+ $firewallPolicyNetRule = New-AzFirewallPolicyNetworkRule -Name $rule.Name -SourceIpGroup $rule.SourceIpGroups -DestinationAddress $rule.DestinationAddresses -DestinationPort $rule.DestinationPorts -Protocol $rule.Protocols`
`131`	`131`	`}`
`132`	`132`	`elseif($rule.DestinationIpGroups)`
`133`	`133`	`{`