-
Notifications
You must be signed in to change notification settings - Fork 541
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
cloudflare_authenticated_origin_pulls per zone certificates setting for enabled cannot be changed #1832
Comments
Thank you for reporting this issue! For maintainers to dig into issues it is required that all issues include the entirety of This issue has been marked with |
I have updated the log output from terraform apply with TF_LOG=DEBUG |
``` // GetOk returns the data for the given key and whether or not the key // has been set to a non-zero value at some point. ``` if enabled is explicitly set to false, then it technically stays at the zero value Resolves cloudflare#1832
``` // GetOk returns the data for the given key and whether or not the key // has been set to a non-zero value at some point. ``` if the tfstate explicitly has `enabled = false`, then it technically stays at the zero value, which means that the second return value from `GetOk` (`ok`) returns false. If `hostname` is an empty string (such as when`cloudflare_authenticated_origin_pulls_certificate` has `type = "per-zone"` (as opposed to `per-hostname`), then those 2 cancel each other out: ``` switch false { case false: // uh oh this evaluates ``` Resolves cloudflare#1832
``` // GetOk returns the data for the given key and whether or not the key // has been set to a non-zero value at some point. ``` if the tfstate explicitly has `enabled = false`, then it technically stays at the zero value, which means that the second return value from `GetOk` (`ok`) returns false. If `hostname` is an empty string (such as when`cloudflare_authenticated_origin_pulls_certificate` has `type = "per-zone"` (as opposed to `per-hostname`), then those 2 cancel each other out: ``` switch false { case false: // uh oh this evaluates ``` ```release-note:bug resource/cloudflare_authenticated_origin_pulls: fix improper handling of enabled=false ``` Resolves cloudflare#1832
This functionality has been released in v3.23.0 of the Terraform Cloudflare Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you! |
Confirmation
Terraform and Cloudflare provider version
Terraform v0.14.3
Your version of Terraform is out of date! The latest version
is 1.2.7. You can update by downloading from https://www.terraform.io/downloads.html
Affected resource(s)
cloudflare_authenticated_origin_pulls
cloudflare_authenticated_origin_pulls_certificate
Terraform configuration files
Debug output
The text was updated successfully, but these errors were encountered: