-
Notifications
You must be signed in to change notification settings - Fork 96
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
CVE November 2024 #4933
CVE November 2024 #4933
Changes from 1 commit
4f36121
74e8fce
01646c3
92536fc
b1d8873
fffd859
5b79497
4debbc7
9dd6035
f218be6
6e94eb4
b48dd47
a9d07ac
6c1b304
efb2a1e
5e9c53c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -67,7 +67,7 @@ lazy val dockerSettings = Seq( | |
dockerBaseImage := "eclipse-temurin:21.0.2_13-jdk-alpine", | ||
dockerRepository := Some("hmda"), | ||
dockerCommands := dockerCommands.value.flatMap { | ||
case cmd@Cmd("FROM",_) => List(cmd, Cmd("RUN", "apk add --no-cache openssl"), | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Remove here since it isn't necessary, the upgrade resolves this cve, although we could use this base image as well: |
||
case cmd@Cmd("FROM",_) => List(cmd, | ||
Cmd("RUN", "apk update"), | ||
Cmd("RUN", "apk upgrade"), | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Resolves CVE-2024-5535, CVE-2024-45492, and CVE-2024-45491 - all critical cves There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This actually resolves the majority of the critical / high cves: CVE-2024-5535 (libcrypto3, libssl3) Current cves:
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Hi @jaredb96 ; would it be worth upgrading the base image at this point? looks like eclipse-temurin:23.0.1_11-jdk-alpine resolves all the critical CVEs (and it also seems to run an "apk update" internally). We could of course try going back a bit farther if we don't want an image pushed yesterday... |
||
Cmd("RUN", "rm /var/cache/apk/*")) | ||
|
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.
Resolves CVE-2023-2976 and CVE-2024-21742