Conor Mongey

@leonardobiffi can you add a test to demonstrate the behaviour that this is fixing?

Can you provide a sample terraform config that reproduces this? Can you provider the output of `TF_LOG=1 terraform apply` ?

@pascalwhoop 🤔 ```tf client_cert = local_file.cert.filename ``` This is incorrect. it should be the contents of the file, and not the location. The commented out value seems correct. ```tf //...

I think this is probably closer to a TLS issue, rather than passing in values from another provider / module. This works for me with 0.2.5 ```tf provider "consul" {}...

@RafaelloLoliop You could leverage the terraform lifecycle [`prevent_destroy`](https://www.terraform.io/docs/language/meta-arguments/lifecycle.html#prevent_destroy) param here ```tf resource "kafka_topic" "logs" { name = "systemd_logs" replication_factor = 2 partitions = 100 lifecycle { prevent_destroy = true }...

@leonardobiffi Would the solution here, not be to remove the ACL from state via `terraform state rm kafka_acl.foo` ?

Reference to how the aws provider does it. https://github.com/hashicorp/terraform-provider-aws/blob/9f3fce2617489a9ec7f3486a51e2981a4410c085/internal/service/s3/bucket.go#L919-L929 I'll take a look at the PR

yay! thanks @muscovitebob 🥇

@benweint Why have special support for setting a bearer token vs just allowing setting of headers? I think we should remove the bearer token, and rename `extra_headers` to `headers` unless...

I've merged #92, thanks for the PR