Upgrade git2go to v33 and libgit2 to 1.3

Tyler Phelan · Tyler Phelan · commit 43644ebb5f30 · 2021-12-07T16:56:50.000-05:00
- run goimports
diff --git a/go.mod b/go.mod
@@ -15,7 +15,7 @@ require (
 	github.com/google/go-containerregistry v0.6.0
 	github.com/google/go-containerregistry/pkg/authn/k8schain v0.0.0-20210610160139-c086c7f16d4e
 	github.com/jinzhu/gorm v1.9.12 // indirect
-	github.com/libgit2/git2go/v31 v31.4.14
+	github.com/libgit2/git2go/v33 v33.0.4
 	github.com/matthewmcnew/archtest v0.0.0-20191014222827-a111193b50ad
 	github.com/mgutz/ansi v0.0.0-20170206155736-9520e82c474b
 	github.com/pkg/errors v0.9.1
diff --git a/go.sum b/go.sum
@@ -1187,8 +1187,8 @@ github.com/lib/pq v1.1.1/go.mod h1:5WUZQaWbwv1U+lTReE5YruASi9Al49XbQIvNi/34Woo=
 github.com/lib/pq v1.8.0/go.mod h1:AlVN5x4E4T544tWzH6hKfbfQvm3HdbOxrmggDNAPY9o=
 github.com/lib/pq v1.10.0/go.mod h1:AlVN5x4E4T544tWzH6hKfbfQvm3HdbOxrmggDNAPY9o=
 github.com/lib/pq v1.10.1/go.mod h1:AlVN5x4E4T544tWzH6hKfbfQvm3HdbOxrmggDNAPY9o=
-github.com/libgit2/git2go/v31 v31.4.14 h1:6GOd3965D9e/+gjxCwZF4eQ+vB9kKB4yKFqdQr6XZ2E=
-github.com/libgit2/git2go/v31 v31.4.14/go.mod h1:c/rkJcBcUFx6wHaT++UwNpKvIsmPNqCeQ/vzO4DrEec=
+github.com/libgit2/git2go/v33 v33.0.4 h1:37xovFBzibhDEdQRLbfWwx3a44JhOIY06UICn2teenc=
+github.com/libgit2/git2go/v33 v33.0.4/go.mod h1:KdpqkU+6+++4oHna/MIOgx4GCQ92IPCdpVRMRI80J+4=
 github.com/lightstep/lightstep-tracer-common/golang/gogo v0.0.0-20190605223551-bc2310a04743/go.mod h1:qklhhLq1aX+mtWk9cPHPzaBjWImj5ULL6C7HFJtXQMM=
 github.com/lightstep/lightstep-tracer-go v0.18.1/go.mod h1:jlF1pusYV4pidLvZ+XD0UBX0ZE6WURAspgAczcDHrL4=
 github.com/lunixbochs/vtclean v0.0.0-20180621232353-2d01aacdc34a/go.mod h1:pHhQNgMf3btfWnGBVipUOjRYhoOsdGqdm/+2c2E2WMI=
diff --git a/pkg/apis/build/v1alpha2/builder_conversion.go b/pkg/apis/build/v1alpha2/builder_conversion.go
@@ -3,6 +3,7 @@ package v1alpha2
 import (
 	"context"
 	"fmt"
+
 	"knative.dev/pkg/apis"
 
 	"github.com/pivotal/kpack/pkg/apis/build/v1alpha1"
diff --git a/pkg/apis/build/v1alpha2/source_resolver_conversion.go b/pkg/apis/build/v1alpha2/source_resolver_conversion.go
@@ -3,6 +3,7 @@ package v1alpha2
 import (
 	"context"
 	"fmt"
+
 	"knative.dev/pkg/apis"
 
 	"github.com/pivotal/kpack/pkg/apis/build/v1alpha1"
diff --git a/pkg/git/certificate_check_callback.go b/pkg/git/certificate_check_callback.go
@@ -1,40 +1,44 @@
 package git
 
 import (
-	git2go "github.com/libgit2/git2go/v31"
-
 	"log"
+
+	git2go "github.com/libgit2/git2go/v33"
+	"github.com/pkg/errors"
 )
 
 func certificateCheckCallback(logger *log.Logger) git2go.CertificateCheckCallback {
-	return func(cert *git2go.Certificate, valid bool, hostname string) git2go.ErrorCode {
+	return func(cert *git2go.Certificate, valid bool, hostname string) error {
 		if valid {
-			return git2go.ErrOk
+			return nil
 		}
 
 		if cert.Kind == git2go.CertificateX509 {
 			if cert.X509 != nil {
 				err := cert.X509.VerifyHostname(hostname)
 				if err != nil {
-					logger.Println("host name could not be verified")
-					return git2go.ErrAuth
+					msg := "host name could not be verified"
+					logger.Println(msg)
+					return errors.Wrap(err, msg)
 				}
 			}
 		} else if cert.Kind == git2go.CertificateHostkey {
 			if cert.Hostkey.Kind == git2go.HostkeyMD5 {
 				if !isByteArrayEmpty(cert.Hostkey.HashMD5[:]) {
-					logger.Println("invalid host key MD5")
-					return git2go.ErrAuth
+					msg := "invalid host key MD5"
+					logger.Println(msg)
+					return errors.New(msg)
 				}
 			} else if cert.Hostkey.Kind == git2go.HostkeySHA1 {
 				if !isByteArrayEmpty(cert.Hostkey.HashSHA1[:]) {
-					logger.Println("invalid host key SHA1")
-					return git2go.ErrAuth
+					msg := "invalid host key SHA1"
+					logger.Println(msg)
+					return errors.New(msg)
 				}
 			}
 		}
 
-		return git2go.ErrorCodeOK
+		return nil
 	}
 
 }
diff --git a/pkg/git/fetch.go b/pkg/git/fetch.go
@@ -6,7 +6,7 @@ import (
 	"path"
 
 	"github.com/BurntSushi/toml"
-	git2go "github.com/libgit2/git2go/v31"
+	git2go "github.com/libgit2/git2go/v33"
 	"github.com/pkg/errors"
 )
 
diff --git a/pkg/git/fetch_test.go b/pkg/git/fetch_test.go
@@ -10,7 +10,7 @@ import (
 	"testing"
 
 	"github.com/BurntSushi/toml"
-	git2go "github.com/libgit2/git2go/v31"
+	git2go "github.com/libgit2/git2go/v33"
 	"github.com/pkg/errors"
 	"github.com/sclevine/spec"
 	"github.com/stretchr/testify/require"
diff --git a/pkg/git/git_keychain.go b/pkg/git/git_keychain.go
@@ -4,7 +4,7 @@ import (
 	"sort"
 	"strings"
 
-	git2go "github.com/libgit2/git2go/v31"
+	git2go "github.com/libgit2/git2go/v33"
 	"github.com/pkg/errors"
 	giturls "github.com/whilp/git-urls"
 	"golang.org/x/crypto/ssh"
diff --git a/pkg/git/git_keychain_test.go b/pkg/git/git_keychain_test.go
@@ -6,7 +6,7 @@ import (
 	"path"
 	"testing"
 
-	git2go "github.com/libgit2/git2go/v31"
+	git2go "github.com/libgit2/git2go/v33"
 	"github.com/sclevine/spec"
 	"github.com/stretchr/testify/require"
 	corev1 "k8s.io/api/core/v1"
diff --git a/pkg/git/k8s_git_keychain_test.go b/pkg/git/k8s_git_keychain_test.go
@@ -8,7 +8,7 @@ import (
 	"encoding/pem"
 	"testing"
 
-	git2go "github.com/libgit2/git2go/v31"
+	git2go "github.com/libgit2/git2go/v33"
 	"github.com/sclevine/spec"
 	"github.com/stretchr/testify/require"
 	v1 "k8s.io/api/core/v1"
diff --git a/pkg/git/remote_git_resolver.go b/pkg/git/remote_git_resolver.go
@@ -7,7 +7,7 @@ import (
 	"os"
 	"strings"
 
-	git2go "github.com/libgit2/git2go/v31"
+	git2go "github.com/libgit2/git2go/v33"
 	"github.com/pkg/errors"
 
 	corev1alpha1 "github.com/pivotal/kpack/pkg/apis/core/v1alpha1"

Original file line number	Diff line number	Diff line change
`@@ -1,40 +1,44 @@`
`1`	`1`	`package git`
`2`	`2`
`3`	`3`	`import (`
`4`		`- git2go "github.com/libgit2/git2go/v31"`
`5`		`-`
`6`	`4`	`"log"`
	`5`	`+`
	`6`	`+ git2go "github.com/libgit2/git2go/v33"`
	`7`	`+ "github.com/pkg/errors"`
`7`	`8`	`)`
`8`	`9`
`9`	`10`	`func certificateCheckCallback(logger *log.Logger) git2go.CertificateCheckCallback {`
`10`		`- return func(cert *git2go.Certificate, valid bool, hostname string) git2go.ErrorCode {`
	`11`	`+ return func(cert *git2go.Certificate, valid bool, hostname string) error {`
`11`	`12`	`if valid {`
`12`		`- return git2go.ErrOk`
	`13`	`+ return nil`
`13`	`14`	`}`
`14`	`15`
`15`	`16`	`if cert.Kind == git2go.CertificateX509 {`
`16`	`17`	`if cert.X509 != nil {`
`17`	`18`	`err := cert.X509.VerifyHostname(hostname)`
`18`	`19`	`if err != nil {`
`19`		`- logger.Println("host name could not be verified")`
`20`		`- return git2go.ErrAuth`
	`20`	`+ msg := "host name could not be verified"`
	`21`	`+ logger.Println(msg)`
	`22`	`+ return errors.Wrap(err, msg)`
`21`	`23`	`}`
`22`	`24`	`}`
`23`	`25`	`} else if cert.Kind == git2go.CertificateHostkey {`
`24`	`26`	`if cert.Hostkey.Kind == git2go.HostkeyMD5 {`
`25`	`27`	`if !isByteArrayEmpty(cert.Hostkey.HashMD5[:]) {`
`26`		`- logger.Println("invalid host key MD5")`
`27`		`- return git2go.ErrAuth`
	`28`	`+ msg := "invalid host key MD5"`
	`29`	`+ logger.Println(msg)`
	`30`	`+ return errors.New(msg)`
`28`	`31`	`}`
`29`	`32`	`} else if cert.Hostkey.Kind == git2go.HostkeySHA1 {`
`30`	`33`	`if !isByteArrayEmpty(cert.Hostkey.HashSHA1[:]) {`
`31`		`- logger.Println("invalid host key SHA1")`
`32`		`- return git2go.ErrAuth`
	`34`	`+ msg := "invalid host key SHA1"`
	`35`	`+ logger.Println(msg)`
	`36`	`+ return errors.New(msg)`
`33`	`37`	`}`
`34`	`38`	`}`
`35`	`39`	`}`
`36`	`40`
`37`		`- return git2go.ErrorCodeOK`
	`41`	`+ return nil`
`38`	`42`	`}`
`39`	`43`
`40`	`44`	`}`
Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,7 @@ import (`
`6`	`6`	`"path"`
`7`	`7`
`8`	`8`	`"github.com/BurntSushi/toml"`
`9`		`- git2go "github.com/libgit2/git2go/v31"`
	`9`	`+ git2go "github.com/libgit2/git2go/v33"`
`10`	`10`	`"github.com/pkg/errors"`
`11`	`11`	`)`
`12`	`12`