Commit bb6f5b9
* {AzureNetwork} fixes Azure#22448 Making priority required
fixes Azure#22448
[This is the API](https://learn.microsoft.com/en-us/rest/api/virtualnetwork/network-security-groups/create-or-update?tabs=HTTP#code-try-0) I invoked.
For the NSG security rules without priorities, seems like the priority parameter is required. Azure portal works as expected. I don't think that's allowed but according to the schema it is because the priority property is not marked as required. I tried to create a NSG security rule without the priorities and it failed with below. Intrestingly the Azure
Request Body:
```
{
"properties": {
"securityRules": [
{
"name": "rule1",
"properties": {
"protocol": "*",
"sourceAddressPrefix": "*",
"destinationAddressPrefix": "*",
"access": "Allow",
"destinationPortRange": "80",
"sourcePortRange": "*",
"direction": "Inbound"
}
}
]
},
"location": "eastus"
}
```
Response received:
```
{
"error": {
"code": "SecurityRuleInvalidPriority",
"message": "Security rule has invalid Priority. Value provided: 0 Allowed range 100-4096.",
"details": []
}
}
```
This PR will make the priority parameter as required.
* Update NetworkSecurityGroupRuleDelete.json
Long running operation should return `azure-AsyncOperation` in response header.
* Update NetworkSecurityGroupDelete.json
Long running operation should return location or azure-AsyncOperation in header.
* Update NetworkSecurityGroupRuleDelete.json
* Update NetworkSecurityGroupDelete.json
* Update NetworkSecurityGroupRuleDelete.json
* Update NetworkSecurityGroupDelete.json
1 parent 9942530 commit bb6f5b9
File tree
3 files changed
+23
-4
lines changed- specification/network/resource-manager/Microsoft.Network/stable/2022-09-01
- examples
3 files changed
+23
-4
lines changedLines changed: 12 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
11 | | - | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
12 | 22 | | |
13 | 23 | | |
Lines changed: 10 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
11 | | - | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
12 | 20 | | |
13 | 21 | | |
14 | 22 | | |
Lines changed: 1 addition & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
820 | 820 | | |
821 | 821 | | |
822 | 822 | | |
| 823 | + | |
823 | 824 | | |
824 | 825 | | |
825 | 826 | | |
| |||
0 commit comments