Skip to content

Commit c3014a3

Browse files
committed
change data structure in PSAzureFirewallWhitelist to string[]
1 parent 768c85c commit c3014a3

File tree

4 files changed

+12
-12
lines changed

4 files changed

+12
-12
lines changed

src/Network/Network.Test/ScenarioTests/AzureFirewallTests.ps1

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1112,15 +1112,15 @@ function Test-AzureFirewallThreatIntelWhitelistCRUD {
11121112

11131113
# Verify
11141114
$getAzureFirewall = Get-AzFirewall -Name $azureFirewallName -ResourceGroupName $rgname
1115-
Assert-AreEqual $threatIntelWhitelist1.FQDNs $getAzureFirewall.ThreatIntelWhitelist.FQDNs
1116-
Assert-AreEqual $threatIntelWhitelist1.IpAddresses $getAzureFirewall.ThreatIntelWhitelist.IpAddresses
1115+
Assert-AreEqual true (-not (Compare-Object $threatIntelWhitelist1.FQDNs $getAzureFirewall.ThreatIntelWhitelist.FQDNs))
1116+
Assert-AreEqual true (-not (Compare-Object $threatIntelWhitelist1.IpAddresses $getAzureFirewall.ThreatIntelWhitelist.IpAddresses))
11171117

11181118
# Modify
11191119
$azureFirewall.ThreatIntelWhitelist = $threatIntelWhitelist2
11201120
Set-AzFirewall -AzureFirewall $azureFirewall
11211121
$getAzureFirewall = Get-AzFirewall -Name $azureFirewallName -ResourceGroupName $rgname
1122-
Assert-AreEqual $threatIntelWhitelist2.FQDNs $getAzureFirewall.ThreatIntelWhitelist.FQDNs
1123-
Assert-AreEqual $threatIntelWhitelist2.IpAddresses $getAzureFirewall.ThreatIntelWhitelist.IpAddresses
1122+
Assert-AreEqual true (-not (Compare-Object $threatIntelWhitelist2.FQDNs $getAzureFirewall.ThreatIntelWhitelist.FQDNs))
1123+
Assert-AreEqual true (-not (Compare-Object $threatIntelWhitelist2.IpAddresses $getAzureFirewall.ThreatIntelWhitelist.IpAddresses))
11241124
}
11251125
finally {
11261126
# Cleanup

src/Network/Network/AzureFirewall/ThreatIntelWhitelist/NewAzureFirewallThreatIntelWhitelistCommand.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,8 @@ public override void Execute()
4141

4242
var threatIntelWhitelist = new PSAzureFirewallThreatIntelWhitelist
4343
{
44-
FQDNs = this.FQDN?.Select(str => str.Trim()).Aggregate((result, item) => result + "," + item),
45-
IpAddresses = this.IpAddress?.Select(str => str.Trim()).Aggregate((result, item) => result + "," + item),
44+
FQDNs = this.FQDN?.Select(str => str.Trim()).ToArray(),
45+
IpAddresses = this.IpAddress?.Select(str => str.Trim()).ToArray(),
4646
};
4747
WriteObject(threatIntelWhitelist);
4848
}

src/Network/Network/Common/NetworkResourceManagerProfile.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1117,8 +1117,8 @@ private static void Initialize()
11171117

11181118
dest.AdditionalProperties = new Dictionary<string, string>()
11191119
{
1120-
{ "ThreatIntel.Whitelist.FQDNs", src.ThreatIntelWhitelist.FQDNs },
1121-
{ "ThreatIntel.Whitelist.IpAddresses", src.ThreatIntelWhitelist.IpAddresses }
1120+
{ "ThreatIntel.Whitelist.FQDNs", src.ThreatIntelWhitelist.FQDNs.Aggregate((result, item) => result + "," + item) },
1121+
{ "ThreatIntel.Whitelist.IpAddresses", src.ThreatIntelWhitelist.IpAddresses.Aggregate((result, item) => result + "," + item) },
11221122
};
11231123
}); ;
11241124
cfg.CreateMap<CNM.PSAzureFirewallSku, MNM.AzureFirewallSku>();
@@ -1138,8 +1138,8 @@ private static void Initialize()
11381138
{
11391139
dest.ThreatIntelWhitelist = new CNM.PSAzureFirewallThreatIntelWhitelist
11401140
{
1141-
FQDNs = src.AdditionalProperties.SingleOrDefault(kvp => kvp.Key.Equals("ThreatIntel.Whitelist.FQDNs")).Value,
1142-
IpAddresses = src.AdditionalProperties.SingleOrDefault(kvp => kvp.Key.Equals("ThreatIntel.Whitelist.IpAddresses")).Value
1141+
FQDNs = src.AdditionalProperties.SingleOrDefault(kvp => kvp.Key.Equals("ThreatIntel.Whitelist.FQDNs")).Value?.Split(',').Select(str => str.Trim()).ToArray(),
1142+
IpAddresses = src.AdditionalProperties.SingleOrDefault(kvp => kvp.Key.Equals("ThreatIntel.Whitelist.IpAddresses")).Value?.Split(',').Select(str => str.Trim()).ToArray()
11431143
};
11441144
}); ;
11451145
cfg.CreateMap<MNM.AzureFirewallSku, CNM.PSAzureFirewallSku>();

src/Network/Network/Models/AzureFirewall/PSAzureFirewallThreatIntelWhitelist.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@ namespace Microsoft.Azure.Commands.Network.Models
1717
{
1818
public class PSAzureFirewallThreatIntelWhitelist
1919
{
20-
public string FQDNs { get; set; }
20+
public string[] FQDNs { get; set; }
2121

22-
public string IpAddresses { get; set; }
22+
public string[] IpAddresses { get; set; }
2323
}
2424
}

0 commit comments

Comments
 (0)