Skip to content

Commit

Permalink
Allow multiple SSL Certificates on SSL Proxy LBs (#4499) (#8478)
Browse files Browse the repository at this point in the history
* mark field as updatable

Co-authored-by: upodroid <[email protected]>

* remove ssl proxy limit

Co-authored-by: upodroid <[email protected]>
Signed-off-by: Modular Magician <[email protected]>

Co-authored-by: upodroid <[email protected]>
  • Loading branch information
modular-magician and upodroid authored Feb 16, 2021
1 parent 7a9d14c commit 8f35afc
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 5 deletions.
3 changes: 3 additions & 0 deletions .changelog/4499.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
```release-note:enhancement
compute: removed max items limit from `google_compute_target_ssl_proxy`. The API currently allows upto 15 Certificates.
```
5 changes: 2 additions & 3 deletions google/resource_compute_target_ssl_proxy.go
Original file line number Diff line number Diff line change
Expand Up @@ -65,9 +65,8 @@ character, which cannot be a dash.`,
Type: schema.TypeList,
Required: true,
Description: `A list of SslCertificate resources that are used to authenticate
connections between users and the load balancer. Currently, exactly
one SSL certificate must be specified.`,
MaxItems: 1,
connections between users and the load balancer. At least one
SSL certificate must be specified.`,
Elem: &schema.Schema{
Type: schema.TypeString,
DiffSuppressFunc: compareSelfLinkOrResourceName,
Expand Down
4 changes: 2 additions & 2 deletions website/docs/r/compute_target_ssl_proxy.html.markdown
Original file line number Diff line number Diff line change
Expand Up @@ -94,8 +94,8 @@ The following arguments are supported:
* `ssl_certificates` -
(Required)
A list of SslCertificate resources that are used to authenticate
connections between users and the load balancer. Currently, exactly
one SSL certificate must be specified.
connections between users and the load balancer. At least one
SSL certificate must be specified.


- - -
Expand Down

0 comments on commit 8f35afc

Please sign in to comment.