Skip to content
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

insecure_ssl = true parameter is not working #352

Closed
ruzickap opened this issue Feb 12, 2020 · 7 comments
Closed

insecure_ssl = true parameter is not working #352

ruzickap opened this issue Feb 12, 2020 · 7 comments
Labels
size/S Status: Up for grabs Issues that are ready to be worked on by anyone Type: Bug Something isn't working as documented

Comments

@ruzickap
Copy link

Terraform Version

$ terraform -v
Terraform v0.12.18

Affected Resource(s)

  • github_repository_webhook
  • (maybe also github_organization_webhook)

Terraform Configuration Files

provider "github" {
  token        = var.prow_oauth_token
  organization = "MyOrg"
}

resource "github_repository_webhook" "prow" {
  repository = "MyRepo"

  configuration {
    url          = "https://prow.myexample.com/hook"
    content_type = "json"
    insecure_ssl = true
    secret       = var.github_webhook_prow_secret
  }

  active = true
  events = ["*"]
}

Actual Behavior

The GitHub Webhook is properly created, but the "SSL Validation" is not disabled.

Steps to Reproduce

I'm using the Terrafrom code mentioned above to create GitHub Webhook.
In the code above you can see the I would like to disable SSL Validation by setting the parameter insecure_ssl = true.

The GitHub Webhook is created by Terraform properly, but the SSL Validation is not disabled - see the screenshot:

image

If I manually click in GitHub Web GUI to "Disable" the webhook is working fine like it should be - but I can not do it via Terraform now.

Thanks for looking at it.

@ruzickap
Copy link
Author

ruzickap commented Feb 17, 2020

Looks like there is only problem with incomplete documentation.

If I use 1 instead of false it's working fine:

insecure_ssl = 1

Or I can use also:

insecure_ssl = 0

@jcudit jcudit added Type: Bug Something isn't working as documented Status: Up for grabs Issues that are ready to be worked on by anyone size/S labels Feb 18, 2020
@vikkyomkar
Copy link
Contributor

@ruzickap @jcudit I will raise a PR for this issue

@jcudit
Copy link
Contributor

jcudit commented Feb 27, 2020

The fix from ☝️ will land in the next release. Thanks again to the community for documenting and fixing this bug. 🙇

@martinssipenko
Copy link
Contributor

We should close this issue as it's fixed by #365

@ruzickap can you do it please?

@ruzickap
Copy link
Author

ruzickap commented Mar 2, 2020

I checked the #365 and I'm missing the change in the documentation: https://github.com/terraform-providers/terraform-provider-github/blob/master/website/docs/index.html.markdown

It should be changed there as well...

@martinssipenko
Copy link
Contributor

@ruzickap the docs already state that boolean should be used (according to examples):
https://www.terraform.io/docs/providers/github/r/repository_webhook.html

@ruzickap
Copy link
Author

ruzickap commented Mar 2, 2020

Ok... closing...

@ruzickap ruzickap closed this as completed Mar 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/S Status: Up for grabs Issues that are ready to be worked on by anyone Type: Bug Something isn't working as documented
Projects
None yet
Development

No branches or pull requests

4 participants