fixes gitlab hook ssl verification

scm/driver/gitlab/repo.go

@@ -113,7 +113,7 @@ func (s *repositoryService) CreateHook(ctx context.Context, repo string, input *
 		params.Set("token", input.Secret)
 	}
 	if input.SkipVerify {
-		params.Set("enable_ssl_verification", "true")
+		params.Set("enable_ssl_verification", "false")
 	}
 	if input.Events.Branch {
 		// no-op

scm/driver/gitlab/repo_test.go

@@ -303,7 +303,6 @@ func TestRepositoryHookCreate(t *testing.T) {
 
 	gock.New("https://gitlab.com").
 		Post("/api/v4/projects/diaspora/diaspora/hooks").
-		MatchParam("enable_ssl_verification", "true").
 		MatchParam("token", "topsecret").
 		MatchParam("url", "https://ci.example.com/hook").
 		Reply(201).
@@ -315,7 +314,7 @@ func TestRepositoryHookCreate(t *testing.T) {
 		Name:       "drone",
 		Target:     "https://ci.example.com/hook",
 		Secret:     "topsecret",
-		SkipVerify: true,
+		SkipVerify: false,
 	}
 
 	client := NewDefault()
@@ -338,6 +337,46 @@ func TestRepositoryHookCreate(t *testing.T) {
 	t.Run("Rate", testRate(res))
 }
 
+func TestRepositoryHookCreate_SkipVerification(t *testing.T) {
+	defer gock.Off()
+
+	gock.New("https://gitlab.com").
+		Post("/api/v4/projects/diaspora/diaspora/hooks").
+		MatchParam("enable_ssl_verification", "false").
+		MatchParam("token", "topsecret").
+		MatchParam("url", "https://ci.example.com/hook").
+		Reply(201).
+		Type("application/json").
+		SetHeaders(mockHeaders).
+		File("testdata/hook_skip_verification.json")
+
+	in := &scm.HookInput{
+		Name:       "drone",
+		Target:     "https://ci.example.com/hook",
+		Secret:     "topsecret",
+		SkipVerify: true,
+	}
+
+	client := NewDefault()
+	got, res, err := client.Repositories.CreateHook(context.Background(), "diaspora/diaspora", in)
+	if err != nil {
+		t.Error(err)
+		return
+	}
+
+	want := new(scm.Hook)
+	raw, _ := ioutil.ReadFile("testdata/hook_skip_verification.json.golden")
+	json.Unmarshal(raw, want)
+
+	if diff := cmp.Diff(got, want); diff != "" {
+		t.Errorf("Unexpected Results")
+		t.Log(diff)
+	}
+
+	t.Run("Request", testRequest(res))
+	t.Run("Rate", testRate(res))
+}
+
 func TestConvertState(t *testing.T) {
 	tests := []struct {
 		src string

scm/driver/gitlab/testdata/hook_skip_verification.json

@@ -0,0 +1,15 @@
+{
+    "id": 1,
+    "url": "http://example.com/hook",
+    "project_id": 3,
+    "push_events": true,
+    "issues_events": true,
+    "merge_requests_events": true,
+    "tag_push_events": true,
+    "note_events": true,
+    "job_events": true,
+    "pipeline_events": true,
+    "wiki_page_events": true,
+    "enable_ssl_verification": false,
+    "created_at": "2012-10-12T17:04:47Z"
+}

scm/driver/gitlab/testdata/hook_skip_verification.json.golden

@@ -0,0 +1,14 @@
+{
+    "ID": "1",
+    "Name": "",
+    "Target": "http://example.com/hook",
+    "Events": [
+        "issues",
+        "tag",
+        "push",
+        "comment",
+        "merge"
+    ],
+    "Active": true,
+    "SkipVerify": true
+}