Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(jans-auth-server): JIT compiler fails under jvm 17 (works perfectly under jvm 11) #5837

Closed
yuriyz opened this issue Aug 11, 2023 · 1 comment · Fixed by #5954
Closed
Assignees
Labels
comp-jans-auth-server Component affected by issue or PR kind-enhancement Issue or PR is an enhancement to an existing functionality kind-feature Issue or PR is a new feature request
Milestone

Comments

@yuriyz
Copy link
Contributor

yuriyz commented Aug 11, 2023

Description

JIT compiler fails under jvm 17 (works perfectly under jvm 11)
We need to find solution for jvm 17.

Caused by: java.lang.reflect.InaccessibleObjectException: Unable to make protected final java.lang.Class java.lang.ClassLoader.defineClass(java.lang.String,byte[],int,int) throws java.lang.ClassFormatError accessible: module java.base does not "opens java.lang" to unnamed module @77b52d12
	at java.base/java.lang.reflect.AccessibleObject.checkCanSetAccessible(AccessibleObject.java:354)
	at java.base/java.lang.reflect.AccessibleObject.checkCanSetAccessible(AccessibleObject.java:297)
	at java.base/java.lang.reflect.Method.checkCanSetAccessible(Method.java:199)
	at java.base/java.lang.reflect.Method.setAccessible(Method.java:193)
	at net.openhft.compiler.CompilerUtils.<clinit>(CompilerUtils.java:65)

Internal packages that will no longer be open by default: https://cr.openjdk.org/~mr/jigsaw/jdk8-packages-denied-by-default

@yuriyz yuriyz added kind-enhancement Issue or PR is an enhancement to an existing functionality comp-jans-auth-server Component affected by issue or PR labels Aug 11, 2023
@yuriyz yuriyz added this to the 1.0.17 milestone Aug 11, 2023
@yuriyz yuriyz self-assigned this Aug 11, 2023
@mo-auto mo-auto added the kind-feature Issue or PR is a new feature request label Aug 11, 2023
@yurem
Copy link
Contributor

yurem commented Aug 11, 2023

With options --add-opens java.base/java.util=ALL-UNNAMED --add-opens java.base/java.lang=ALL-UNNAMED --add-opens java.base/com.sun=ALL-UNNAMED --add-opens java.base/com.sun.tools.javac.file=ALL-UNNAMED --illegal-access=permit it also fails.

We need to find JDK17 solution because in future versions --add-opens developers can deprecate at all.

yuriyz added a commit that referenced this issue Sep 5, 2023
#5837
Signed-off-by: YuriyZ <yzabrovarniy@gmail.com>
yuriyz added a commit that referenced this issue Sep 5, 2023
devrimyatar pushed a commit that referenced this issue Dec 30, 2023
#5837

Signed-off-by: YuriyZ <yzabrovarniy@gmail.com>
Signed-off-by: Mustafa Baser <mbaser@mail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
comp-jans-auth-server Component affected by issue or PR kind-enhancement Issue or PR is an enhancement to an existing functionality kind-feature Issue or PR is a new feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants