search

dahendel / terraform-provider-awx2

Terraform 0.12.x compatible awx/tower provider
Mozilla Public License 2.0
6 stars 2 forks source link

Name of ssl_verify option is misleading #4

Open davidfischer-ch opened 4 years ago

davidfischer-ch commented 4 years ago

The option ssl_verify when set to true sounds like verification is enabled, and false, sounds like skipped. In reality, this is the opposite!

The provider option should be named ssl_skip_verify or the value inverted in https://github.com/dahendel/terraform-provider-awx2/blob/master/awx/config.go#L21.

dahendel commented 4 years ago

I agree. I will work on that with work to sync the repositories.

davidfischer-ch commented 4 years ago

If I am not mistaking this is fixed in my fork.