-
Notifications
You must be signed in to change notification settings - Fork 625
Add Java version check in server script and handle java9.option exclusions script changes #33671
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
base: integration
Are you sure you want to change the base?
Add Java version check in server script and handle java9.option exclusions script changes #33671
Conversation
…cript-for-java-24' into add-java-level-verified
|
Created this new PR with verified commits Hello @jhanders34 , |
|
!build (view Open Liberty Personal Build - ❌ completed with errors/failures.) Note: Target locations of links might be accessible only to IBM employees. |
Code analysis and actionsDO NOT DELETE THIS COMMENT.
|
release buglabel if applicable: https://github.com/OpenLiberty/open-liberty/wiki/Open-Liberty-Conventions).Fixes ##32996
We include the
--add-exports java.base/sun.security.action=ALL-UNNAMEDJVM parameter in the java9.options file that is needed for the com.ibm.crypto.ibmkeycert.jar file.Starting in Java 24, the sun.security.action.* classes were removed, so if the JVM parameter
--add-exports java.base/sun.security.action=ALL-UNNAMEDis passed in, the JVM gives an informational warning:in the console.log file.
To fix this problem, the
--add-exports java.base/sun.security.action=ALL-UNNAMEDparameter should be excluded whenever running on Java 24+.This update checks for the Java level in the release file (if it exists) in the JAVA_HOME directory. From this, if we are able to determine the Java level being used is Java 24+, then we exclude the JVM parameter
--add-exports java.base/sun.security.action=ALL-UNNAMED. This update will also allow future JVM parameters from the java9.options file to be excluded if necessary.This PR also grants explicit JNI permission when starting the JVM on z/OS when running on Java 24+. JNI is required on the z/OS platform and in Java 24+, explicit permission when starting the JVM is required to use JNI.