Skip to content

Commit

Permalink
Fix S360 swagger error, certificateId not required (#22339)
Browse files Browse the repository at this point in the history
* Fix S360 swagger error, certificateId not required

Fix S360 swagger error, certificateId not required per service behavior

* Update ContainerApps_ListCustomHostNameAnalysisErrorResponse.json

fix example error

---------

Co-authored-by: Nan Jiang <naji@microsoft.com>
  • Loading branch information
najian and Nan Jiang authored Feb 3, 2023
1 parent 9f33c53 commit 638734a
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 13 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -737,8 +737,7 @@
"description": "Custom Domain of a Container App",
"type": "object",
"required": [
"name",
"certificateId"
"name"
],
"properties": {
"name": {
Expand All @@ -758,7 +757,7 @@
}
},
"certificateId": {
"description": "Resource Id of the Certificate to be bound to this hostname. Must exist in the Managed Environment.",
"description": "Resource Id of the Certificate to be bound to this hostname.",
"type": "string"
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -860,8 +860,7 @@
"description": "Custom Domain of a Container App",
"type": "object",
"required": [
"name",
"certificateId"
"name"
],
"properties": {
"name": {
Expand All @@ -881,7 +880,7 @@
}
},
"certificateId": {
"description": "Resource Id of the Certificate to be bound to this hostname. Must exist in the Managed Environment.",
"description": "Resource Id of the Certificate to be bound to this hostname.",
"type": "string"
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -741,8 +741,7 @@
"description": "Custom Domain of a Container App",
"type": "object",
"required": [
"name",
"certificateId"
"name"
],
"properties": {
"name": {
Expand All @@ -762,7 +761,7 @@
}
},
"certificateId": {
"description": "Resource Id of the Certificate to be bound to this hostname. Must exist in the Managed Environment.",
"description": "Resource Id of the Certificate to be bound to this hostname.",
"type": "string"
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
"resourceGroupName": "rg",
"containerAppName": "testcontainerApp0",
"customHostname": "my.name.corp",
"api-version": "2022-03-03"
"api-version": "2022-03-01"
},
"responses": {
"200": {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -882,8 +882,7 @@
"description": "Custom Domain of a Container App",
"type": "object",
"required": [
"name",
"certificateId"
"name"
],
"properties": {
"name": {
Expand All @@ -903,7 +902,7 @@
}
},
"certificateId": {
"description": "Resource Id of the Certificate to be bound to this hostname. Must exist in the Managed Environment.",
"description": "Resource Id of the Certificate to be bound to this hostname.",
"type": "string"
}
}
Expand Down

0 comments on commit 638734a

Please sign in to comment.