Skip to content

Changes to support launch on java 24 or higher. This is based on the changes of PR #969#994

Closed
akuhtz wants to merge 1 commit into
AdoptOpenJDK:masterfrom
akuhtz:java24-fix
Closed

Changes to support launch on java 24 or higher. This is based on the changes of PR #969#994
akuhtz wants to merge 1 commit into
AdoptOpenJDK:masterfrom
akuhtz:java24-fix

Conversation

@akuhtz

@akuhtz akuhtz commented Feb 17, 2026

Copy link
Copy Markdown
Contributor

Changes to support launch on java 24 or higher. This is based on the changes of PR #969

@akuhtz akuhtz changed the title Changes to support launch on java 24 or higher. This is based on the changes of PR 969 Changes to support launch on java 24 or higher. This is based on the changes of PR #969 Feb 17, 2026
akuhtz added a commit to akuhtz/OpenWebStart that referenced this pull request Feb 17, 2026
@sonarqubecloud

sonarqubecloud Bot commented May 7, 2026

Copy link
Copy Markdown

if (security != null) {
return security.getApplication();
}
return null;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is problematic for Jnlp Services.

@janakmulani

Copy link
Copy Markdown
Contributor

PR #969 has some issues. Moreover, the changes related to the SecurityManager also have some issues. Will create a new branch and new PR after testing various scenarios. This will need time and effort, hence won't be part of the next release (due likely in June).

@akuhtz

akuhtz commented May 27, 2026

Copy link
Copy Markdown
Contributor Author

@janakmulani Please add reference to this PR with your new solution.

@guykatz

guykatz commented Jun 15, 2026

Copy link
Copy Markdown

very critical for me also. @akuhtz since you opened a PR to solve this, and assuming I dont really care too much about the things that @janakmulani stated as problematic, is there a bypass i can use if I need tumerin 25 support right now?
thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants