-
Notifications
You must be signed in to change notification settings - Fork 465
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
[WFCORE-7064] Do not set SM options on JDK24+ #6252
Conversation
The test changes look good, but I'm wondering about the production code changes. We don't want to be setting -Djava.security.manager=allow on our own on SE 24+, but that would be done in things like the scripts or the launcher. The code here seems to be about processing external input. (It may be input that's been through various layers and transformations, but ultimately it's derived from external input.) If we get input to enable the SM, I don't think we should ignore it on 24; we should process it and let things fail. |
host-controller/src/main/java/org/jboss/as/host/controller/Main.java
Outdated
Show resolved
Hide resolved
Core -> Full Integration Build 14053 outcome was FAILURE using a merge of dc30e56 Failed tests
|
Thinking about it you are of course right @bstansberry ! I updated PR to include only test suite changes. |
Thanks, @ropalka! |
https://issues.redhat.com/browse/WFCORE-7064