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

The dependency golang.org/x/text v0.3.2 has a CVE issued against it #6

Closed
karlmutch opened this issue Feb 25, 2021 · 4 comments · Fixed by #8
Closed

The dependency golang.org/x/text v0.3.2 has a CVE issued against it #6

karlmutch opened this issue Feb 25, 2021 · 4 comments · Fixed by #8

Comments

@karlmutch
Copy link
Contributor

Hi,

When using the github security checking tools the golang.org/x/text v0.3.2 dependency causes an issue to be raised.

https://nvd.nist.gov/vuln/detail/CVE-2020-14040

https://snyk.io/vuln/SNYK-GOLANG-GOLANGORGXTEXTENCODINGUNICODE-609611

Thanks for this project I am finding it very useful,
Karl

@bzz
Copy link
Member

bzz commented Apr 1, 2021

Thank you for rising it @karlmutch !

Thanks for this project I am finding it very useful,

Glad to hear!

Did I get it right that the resolution path is to upgrade golang.org/x/text/encoding/unicode to version 0.3.3 or higher? A PR would be very welcome ;)

@karlmutch
Copy link
Contributor Author

Thanks.

PR Added at #8

There appeared to be some issues in the github based testing which don't occur when I use stock go test. Should these be tagged as false positives ?

Karl

@bzz bzz closed this as completed in #8 Apr 22, 2021
@bzz
Copy link
Member

bzz commented Apr 22, 2021

Indeed, I have fixed one for the CI profiles in #12 and CI build on windows needs further investigation and does not seem to be related to the changes.

Thank you for the fix!

@vmarkovtsev
Copy link
Collaborator

Thank you for maintaining the project @bzz ❤️! I wish I could devote some time... Let's hope for the summer.

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

Successfully merging a pull request may close this issue.

3 participants