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.

Sign up for GitHub

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

Jump to bottom

Remove Java 7 support #895

Open

elefeint opened this issue Aug 8, 2022 · 0 comments

Labels

cleanup p3 Tech Debt

Contributor

elefeint commented Aug 8, 2022 •

edited

Loading

Follow-up to #893

The core plugin still refers to isSandboxEnforced() which can be deleted entirely for java7 which is not supported (everything with java7 string can be deleted including tests

As a side effect, these flags can be removed:
-Duse_jetty9_runtime=true -D--enable_all_permissions=true
as we are only on jetty9 now, and we do not have a security manager for java8, 11 and 17.

cc: @ludoch -- follow-up issue to remove Java 7 logic.

elefeint added Tech Debt p3 labels

emmileaf added the cleanup label

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