Skip to content

Commit 888d8b8

Browse files
author
Jeremy Udit
authored
Merge pull request integrations#365 from vikkyomkar/issue-352
Fixed : insecure_ssl parameter behaviour in webhook creation
2 parents 98f88de + a9dd7e9 commit 888d8b8

File tree

3 files changed

+6
-6
lines changed

3 files changed

+6
-6
lines changed

github/resource_github_organization_webhook_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ func TestAccGithubOrganizationWebhook_basic(t *testing.T) {
3232
Configuration: map[string]interface{}{
3333
"url": "https://google.de/webhook",
3434
"content_type": "json",
35-
"insecure_ssl": "true",
35+
"insecure_ssl": true,
3636
},
3737
Active: true,
3838
}),
@@ -47,7 +47,7 @@ func TestAccGithubOrganizationWebhook_basic(t *testing.T) {
4747
Configuration: map[string]interface{}{
4848
"url": "https://google.de/webhooks",
4949
"content_type": "form",
50-
"insecure_ssl": "false",
50+
"insecure_ssl": false,
5151
},
5252
Active: false,
5353
}),

github/resource_github_repository_webhook_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ func TestAccGithubRepositoryWebhook_basic(t *testing.T) {
3333
Configuration: map[string]interface{}{
3434
"url": "https://google.de/webhook",
3535
"content_type": "json",
36-
"insecure_ssl": "true",
36+
"insecure_ssl": true,
3737
},
3838
Active: true,
3939
}),
@@ -48,7 +48,7 @@ func TestAccGithubRepositoryWebhook_basic(t *testing.T) {
4848
Configuration: map[string]interface{}{
4949
"url": "https://google.de/webhooks",
5050
"content_type": "form",
51-
"insecure_ssl": "false",
51+
"insecure_ssl": false,
5252
},
5353
Active: false,
5454
}),
@@ -84,7 +84,7 @@ func TestAccGithubRepositoryWebhook_secret(t *testing.T) {
8484
"url": "https://www.terraform.io/webhook",
8585
"content_type": "json",
8686
"secret": "********",
87-
"insecure_ssl": "false",
87+
"insecure_ssl": false,
8888
},
8989
Active: true,
9090
}),

github/schema_webhook_configuration.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ func webhookConfigurationSchema() *schema.Schema {
2525
Sensitive: true,
2626
},
2727
"insecure_ssl": {
28-
Type: schema.TypeString,
28+
Type: schema.TypeBool,
2929
Optional: true,
3030
},
3131
},

0 commit comments

Comments
 (0)