Skip to content
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

Remove deprecated java.security.AccessControl #2672

Merged
merged 1 commit into from
Apr 9, 2024

Conversation

pks-1981
Copy link
Contributor

@pks-1981 pks-1981 commented Apr 7, 2024

Remove deprecated java.security.AccessControl

@pks-1981 pks-1981 requested a review from a team as a code owner April 7, 2024 07:01
@dieppa dieppa self-assigned this Apr 9, 2024
@dieppa
Copy link
Contributor

dieppa commented Apr 9, 2024

Thanks for your contribution.

Having a look to the official JEP 411: Deprecate the Security Manager for Removal, which says explicitly :

Deprecate the Security Manager for removal in a future release. The Security Manager dates from Java 1.0. It has not been the primary means of securing client-side Java code for many years, and it has rarely been used to secure server-side code. To move Java forward, we intend to deprecate the Security Manager for removal in concert with the legacy Applet API (JEP 398).

It provides an extensive explanation on the motivations and implications, but it can be summarised with:

The Security manager's goal was to protect agains two types of vulnerabilities: malicious intent by remote code and the threat of accidental vulnerabilities in local code. The threat of the first was mainly a concern with applets. Now applet are not supported and , so the protection from the Security Manager is no longer significant in this matter.
Regarding the threat of accidental vulnerabilities in local code, it's almost impossible to address with the Security Manager. Many of the claims that the Security Manager is widely used to secure local code do not stand up to scrutiny; it is used far less in production than many people assume.

From all the above, we have decide to give green light to this change 😃

@dieppa dieppa merged commit e7cf364 into wiremock:master Apr 9, 2024
10 checks passed
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.

None yet

2 participants