-
Notifications
You must be signed in to change notification settings - Fork 1.4k
CVEs: Release-1.8 and Release-1.9 have CVEs that can only be resolved by upgrading to Go 1.23 #12173
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
Comments
I ran govulncheck with verbose output against the release-1.8 branch and got the following:
Meanwhile the trivy scan found these vulnerabilities, but does not tell you if they're called or not:
CVE-2025-22869 is the same as GO-2025-3487. So I believe the question is now this: Since |
I would say yes. I think if govulncheck determines we don't use the affected code we can't justify the breaking change of a go minor version bump in go.mod on release branches |
Yes we trust govulncheck. Otherwise we should never have added it. |
@chrischdi Do you remember if we ever excluded CVE's in trivy and how to do it? (so we can get the action back to green) |
Yes, for trivy we can add a file like this: |
Great! @cprivitere I think we can go ahead and add a .trivyignore file on the branches where it's needed |
/triage accepted |
/area security |
/kind failing-test |
/priority important-soon |
/close |
@cprivitere: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
The weekly security scan task has found two modules that have CVEs in our codebase.
To bump these to the fixed version (0.35 for crypto and 0.38 for net), we would need to bump the project to use 1.23. We don't generally do this as it'd force anyone consuming the module downstream to upgrade their go as well and that might not work for their projects or timelines.
This issue is to decide and document whether we should bump 1.8 and 1.9 to Go 1.23 or leave it as is.
The text was updated successfully, but these errors were encountered: