Merge pull request Azure#10389 from Khushboo-Baheti/BastionFix

wyunchi-ms · web-flow · commit 4a96e0c33ece · 2019-10-25T09:21:31.000+08:00
Subnet name AzureBastionSubnet should be case insensitive
diff --git a/src/Network/Network/ChangeLog.md b/src/Network/Network/ChangeLog.md
@@ -20,6 +20,7 @@
 
 ## Upcoming Release
 * Fix incorrect parameter description in `New-AzApplicationGatewaySku` reference documentation
+* Fix required subnet with name AzureBastionSubnet in `PSBastion` can be case insensitive
 
 ## Version 1.15.0
 * Add new cmdlet Get-AzAvailableServiceAlias which can be called to get the aliases that can be used for Service Endpoint Policies.
diff --git a/src/Network/Network/Models/Bastion/PSBastion.cs b/src/Network/Network/Models/Bastion/PSBastion.cs
@@ -53,7 +53,7 @@ public void Allocate(PSVirtualNetwork virtualNetwork, PSPublicIpAddress publicIp
             PSSubnet BastionSubnet = null;
             try
             {
-                BastionSubnet = virtualNetwork.Subnets.Single(subnet => BastionSubnetName.Equals(subnet.Name));
+                BastionSubnet = virtualNetwork.Subnets.Single(subnet => BastionSubnetName.Equals(subnet.Name, StringComparison.OrdinalIgnoreCase));
             }
 
             catch (InvalidOperationException)

Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,7 @@ public void Allocate(PSVirtualNetwork virtualNetwork, PSPublicIpAddress publicIp`
`53`	`53`	`PSSubnet BastionSubnet = null;`
`54`	`54`	`try`
`55`	`55`	`{`
`56`		`- BastionSubnet = virtualNetwork.Subnets.Single(subnet => BastionSubnetName.Equals(subnet.Name));`
	`56`	`+ BastionSubnet = virtualNetwork.Subnets.Single(subnet => BastionSubnetName.Equals(subnet.Name, StringComparison.OrdinalIgnoreCase));`
`57`	`57`	`}`
`58`	`58`
`59`	`59`	`catch (InvalidOperationException)`