Skip to content
This repository has been archived by the owner on Apr 9, 2024. It is now read-only.

Update dependencies for gokart and to address panic #91

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

jauderho
Copy link

Building gokart as is resulted in a panic.

panic: no concrete method: func (*crypto/x509.Certificate).CheckCRLSignature(crl *crypto/x509/pkix.CertificateList) error

goroutine 1 [running]:
golang.org/x/tools/go/ssa.(*Program).declaredFunc(0xc0001ae6c0, 0xc0001f33e0)
	golang.org/x/tools@v0.1.12/go/ssa/methods.go:149 +0xf9
golang.org/x/tools/go/ssa.(*Program).originFunc(0xc0005e6cd0?, 0x40f827?)
	golang.org/x/tools@v0.1.12/go/ssa/source.go:193 +0x32
golang.org/x/tools/go/ssa.(*Program).addMethod(0xc0001ae6c0, 0xc00096d0a0, 0xc0005e6cd0, 0xa988e0?)
	golang.org/x/tools@v0.1.12/go/ssa/methods.go:109 +0x179
golang.org/x/tools/go/ssa.(*Program).needMethods(0xc0001ae6c0, {0xc92798?, 0xc0003941b0?}, 0x0, 0x106f760?)
	golang.org/x/tools@v0.1.12/go/ssa/methods.go:200 +0x705
golang.org/x/tools/go/ssa.(*Program).needMethods(0xc0001ae6c0, {0xc927e8?, 0xc0003941c0?}, 0x0, 0x106f768?)
	golang.org/x/tools@v0.1.12/go/ssa/methods.go:223 +0x2d7
golang.org/x/tools/go/ssa.(*Program).needMethods(0xc0001ae6c0, {0xc92810?, 0xc000191c20?}, 0x1, 0x106f760?)
	golang.org/x/tools@v0.1.12/go/ssa/methods.go:255 +0x617
golang.org/x/tools/go/ssa.(*Program).needMethods(0xc0001ae6c0, {0xc92770?, 0xc000182f50?}, 0x0, 0x106f760?)
	golang.org/x/tools@v0.1.12/go/ssa/methods.go:248 +0x487
golang.org/x/tools/go/ssa.(*Program).needMethods(0xc0001ae6c0, {0xc92838?, 0xc0001e6840?}, 0x0, 0x106f760?)
	golang.org/x/tools@v0.1.12/go/ssa/methods.go:260 +0x5ae
golang.org/x/tools/go/ssa.(*Program).needMethods(0xc0001ae6c0, {0xc927c0?, 0xc0001dfdc0?}, 0x0, 0x106f768?)
	golang.org/x/tools@v0.1.12/go/ssa/methods.go:236 +0x3bc
golang.org/x/tools/go/ssa.(*Program).needMethods(0xc0001ae6c0, {0xc92810?, 0xc000191ce0?}, 0x1, 0x106f760?)
	golang.org/x/tools@v0.1.12/go/ssa/methods.go:255 +0x617
golang.org/x/tools/go/ssa.(*Program).needMethods(0xc0001ae6c0, {0xc92770?, 0xc000182a10?}, 0x0, 0x106f760?)
	golang.org/x/tools@v0.1.12/go/ssa/methods.go:248 +0x487
golang.org/x/tools/go/ssa.(*Program).needMethods(0xc0001ae6c0, {0xc92798?, 0xc000394310?}, 0x0, 0x106f760?)
	golang.org/x/tools@v0.1.12/go/ssa/methods.go:220 +0x29b
golang.org/x/tools/go/ssa.(*Program).needMethods(0xc0001ae6c0, {0xc92838?, 0xc0001e68b8?}, 0x0, 0x106f760?)
	golang.org/x/tools@v0.1.12/go/ssa/methods.go:260 +0x5ae
golang.org/x/tools/go/ssa.(*Program).needMethods(0xc0001ae6c0, {0xc92770?, 0xc0001829a0?}, 0x0, 0x106f760?)
	golang.org/x/tools@v0.1.12/go/ssa/methods.go:208 +0x1d0
golang.org/x/tools/go/ssa.(*Program).needMethods(0xc0001ae6c0, {0xc92838?, 0xc0001e7080?}, 0x0, 0x106f760?)
	golang.org/x/tools@v0.1.12/go/ssa/methods.go:260 +0x5ae
golang.org/x/tools/go/ssa.(*Program).needMethods(0xc0001ae6c0, {0xc92798?, 0xc0007c52c0?}, 0x0, 0x106f760?)
	golang.org/x/tools@v0.1.12/go/ssa/methods.go:208 +0x1d0
golang.org/x/tools/go/ssa.(*Program).needMethods(0xc0001ae6c0, {0xc92838?, 0xc0007b4cf0?}, 0x0, 0x106f760?)
	golang.org/x/tools@v0.1.12/go/ssa/methods.go:260 +0x5ae
golang.org/x/tools/go/ssa.(*Program).needMethods(0xc0001ae6c0, {0xc92798?, 0xc000865290?}, 0x0, 0x106f760?)
	golang.org/x/tools@v0.1.12/go/ssa/methods.go:208 +0x1d0
golang.org/x/tools/go/ssa.(*Program).needMethods(0xc0001ae6c0, {0xc92770?, 0xc0007e6070?}, 0x0, 0x9f84f6?)
	golang.org/x/tools@v0.1.12/go/ssa/methods.go:242 +0x458
golang.org/x/tools/go/ssa.(*Program).needMethodsOf(0xc0001ae6c0, {0xc92770?, 0xc0007e6070?}, 0xc000853da0?)
	golang.org/x/tools@v0.1.12/go/ssa/methods.go:172 +0x7f
golang.org/x/tools/go/ssa.(*Package).build(0xc0000b3580)
	golang.org/x/tools@v0.1.12/go/ssa/builder.go:2427 +0x133
sync.(*Once).doSlow(0xc0001ae6c0?, 0xc0005e19f0?)
	sync/once.go:74 +0xc2
sync.(*Once).Do(...)
	sync/once.go:65
golang.org/x/tools/go/ssa.(*Package).Build(...)
	golang.org/x/tools@v0.1.12/go/ssa/builder.go:2413
golang.org/x/tools/go/analysis/passes/buildssa.run(0xc000978f70)
	golang.org/x/tools@v0.1.12/go/analysis/passes/buildssa/buildssa.go:73 +0x1a8
github.com/praetorian-inc/gokart/run.RunAnalyzers({0x109efe0, 0x5, 0xc000159a78?}, 0xc0003a6480)
	github.com/praetorian-inc/gokart@v0.5.1/run/run.go:133 +0x1cf
github.com/praetorian-inc/gokart/run.Run({0x109efe0, 0x5, 0x5}, {0xc00018f090?, 0x0?, 0x0?})
	github.com/praetorian-inc/gokart@v0.5.1/run/run.go:42 +0x125
github.com/praetorian-inc/gokart/analyzers.Scan({0xc00018f090?, 0x1, 0x1})
	github.com/praetorian-inc/gokart@v0.5.1/analyzers/scan.go:163 +0x5d1
github.com/praetorian-inc/gokart/cmd.glob..func1(0x10a67c0?, {0xc00018f090, 0x1, 0x1})
	github.com/praetorian-inc/gokart@v0.5.1/cmd/scan.go:91 +0x425
github.com/spf13/cobra.(*Command).execute(0x10a67c0, {0xc00018f060, 0x1, 0x1})
	github.com/spf13/cobra@v1.2.1/command.go:860 +0x663
github.com/spf13/cobra.(*Command).ExecuteC(0x10a6540)
	github.com/spf13/cobra@v1.2.1/command.go:974 +0x3bd
github.com/spf13/cobra.(*Command).Execute(...)
	github.com/spf13/cobra@v1.2.1/command.go:902
github.com/praetorian-inc/gokart/cmd.Execute(...)
	github.com/praetorian-inc/gokart@v0.5.1/cmd/root.go:61
main.main()
	github.com/praetorian-inc/gokart@v0.5.1/main.go:38 +0x25

Updating using go get -u and rebuilding results in a working binary.

This PR includes the updated dependencies and also adds Dependabot support to keep packages and workflows up to date via PRs (as needed)

Signed-off-by: Jauder Ho <jauderho@users.noreply.github.com>
@ryantm
Copy link

ryantm commented May 20, 2023

I tried using this branch and it worked for me. Thanks!

dependabot bot and others added 5 commits July 24, 2023 12:21
Bumps [github.com/go-git/go-git/v5](https://github.com/go-git/go-git) from 5.6.1 to 5.8.0.
- [Release notes](https://github.com/go-git/go-git/releases)
- [Commits](go-git/go-git@v5.6.1...v5.8.0)

---
updated-dependencies:
- dependency-name: github.com/go-git/go-git/v5
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [actions/checkout](https://github.com/actions/checkout) from 2 to 3.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v2...v3)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [goreleaser/goreleaser-action](https://github.com/goreleaser/goreleaser-action) from 2 to 4.
- [Release notes](https://github.com/goreleaser/goreleaser-action/releases)
- [Commits](goreleaser/goreleaser-action@v2...v4)

---
updated-dependencies:
- dependency-name: goreleaser/goreleaser-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [actions/setup-go](https://github.com/actions/setup-go) from 2 to 4.
- [Release notes](https://github.com/actions/setup-go/releases)
- [Commits](actions/setup-go@v2...v4)

---
updated-dependencies:
- dependency-name: actions/setup-go
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Jauder Ho <jauderho@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants