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

upgrade go dependencies to fix indirectly imported CVE #42

Closed
cboitel opened this issue Nov 19, 2021 · 3 comments
Closed

upgrade go dependencies to fix indirectly imported CVE #42

cboitel opened this issue Nov 19, 2021 · 3 comments

Comments

@cboitel
Copy link

cboitel commented Nov 19, 2021

While reviewing dependencies on one of our project, i found the following

  1. Extract from go mod graph:
github.com/hashicorp/[email protected] github.com/miekg/[email protected]
github.com/hashicorp/[email protected] github.com/hashicorp/[email protected]
github.com/hashicorp/consul/[email protected] github.com/hashicorp/[email protected]
github.com/bketelsen/[email protected] github.com/hashicorp/consul/[email protected]
  1. github.com/miekg/[email protected] suffers a CVE which is fixed in later versions which are now imported in latest version of github.com/hashicorp/consul/api (v1.14.0)

Upgrading to latest official version of dependencies would fix the issues. I will report to other projects using this module as dependency so they can track the upgrade once it is available here.

@bketelsen
Copy link
Owner

v0.0.5 released @cboitel @umarcor

@umarcor
Copy link

umarcor commented Nov 21, 2021

Thanks @bketelsen!

Unfortunately, this issue seems to remain:

# go mod graph | $GOPATH/bin/gomodtree github.com/miekg/[email protected]
github.com/miekg/[email protected]
|   github.com/hashicorp/[email protected]
|   |   github.com/hashicorp/[email protected]
|   |   |   github.com/hashicorp/consul/[email protected]
|   |   |   |   github.com/bketelsen/crypt

Which results in:

The issue was fixed in hashicorp's repos already:

Therefore, bumping to v1.11.0-beta3 should fix the issue. However:

# go get github.com/hashicorp/consul/[email protected]
go: downloading github.com/hashicorp/consul v1.11.0-beta3
go get: module github.com/hashicorp/[email protected] found, but does not contain package github.com/hashicorp/consul/api

Do we need to wait until v1.11.0 is tagged in hashicorp/consul?

/cc @rboyer

@umarcor
Copy link

umarcor commented Nov 21, 2021

According to https://pkg.go.dev/github.com/hashicorp/consul/api?tab=versions, v1.11.0 of github.com/hashicorp/consul/api was released on Sep 2021. So, we don't need to look at the regular tags in github.com/hashicorp/consul. Instead: https://github.com/hashicorp/consul/blob/api/v1.11.0/api/go.mod.

@rboyer maybe you can create tag api/v1.11.1?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants