-
Notifications
You must be signed in to change notification settings - Fork 186
Fix CVE-2025-58057 #4338
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
Fix CVE-2025-58057 #4338
Conversation
|
Nice helper script ;) You can make the changes on the root of the build file like this which will iterate over the subprojects That way we dont have to wackamole the build file in each subproject |
Good suggestion! I will apply this |
build.gradle
Outdated
| // Force spotless depending on newer version of guava due to CVE-2023-2976. Remove after spotless upgrades. | ||
| resolutionStrategy.force "com.google.guava:guava:32.1.3-jre" | ||
| resolutionStrategy.force 'org.apache.commons:commons-compress:1.26.0' | ||
| resolutionStrategy.force 'io.netty:netty-buffer:4.1.125.Final' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we really using all these?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to include those netty modules, otherwise the build will fail
Signed-off-by: Nathalie Jonathan <nathhjo@amazon.com>
Signed-off-by: Nathalie Jonathan <nathhjo@amazon.com>
Signed-off-by: Nathalie Jonathan <nathhjo@amazon.com>
6d97145 to
1753f60
Compare
* fix cve Signed-off-by: Nathalie Jonathan <nathhjo@amazon.com> * Force version for other netty modules Signed-off-by: Nathalie Jonathan <nathhjo@amazon.com> * Apply suggestion Signed-off-by: Nathalie Jonathan <nathhjo@amazon.com> * Use versions.netty Signed-off-by: Nathalie Jonathan <nathhjo@amazon.com> --------- Signed-off-by: Nathalie Jonathan <nathhjo@amazon.com> (cherry picked from commit eec7179)
* fix cve * Force version for other netty modules * Apply suggestion * Use versions.netty --------- (cherry picked from commit eec7179) Signed-off-by: Nathalie Jonathan <nathhjo@amazon.com> Co-authored-by: Nathalie Jonathan <143617992+nathaliellenaa@users.noreply.github.com>
Description
Fix CVE-2025-58057
Force
netty-codecto 4.1.125.FinalCouldn't find dependency insight for
netty-codec-compressionRelated Issues
Resolves #[Issue number to be closed when this PR is merged]
Check List
--signoff.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.