-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Bump GraalVM and Mandrel version to 22.1 #25366
Conversation
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.
The svm thing is a bit concerning but let's get this in.
Does it make sense to push the envelope a bit and backport it to 2.9?
Yes, it make sense to backport it to 2.9, GraalVM 22.0.0.2 might be affected by the Java ECDSA Signature Vulnerability so if CI pass with all the battery of tests that Quarkus has, then it's a sane choice. We are already using GraalVM 22.1 with Quarkus 2.8.2 and it compiles fine for our use case (through we haven't extensively tested it yet) and having an official support by Quarkus it will give us more confidence to use it. Also, the |
That makes sense: @zakkak we should probably have a specific version for |
ACK, that makes sense. For some context though let me clarify that the only reason Quarkus depends on Update: I forgot to also mention that as a result using an older graal-sdk during non-native compilation doesn't expose Quarkus applications to https://security.snyk.io/vuln/SNYK-JAVA-ORGGRAALVMSDK-2767964 (which AFAIK only affects the enterprise edition of GraalVM, see https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2022-21449) |
I have been testing I am +0 on it. |
Closes #25199
Openning as Draft till CI runs on my fork