Skip to content

[GR-57827] Initialize security providers at run time. #10143

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
Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

[GR-57827] Initialize security providers at run time. #10143

wants to merge 4 commits into from
+422 −289

Conversation

graalvmbot
Copy link
Collaborator

The code walkthrough:

  • Remove fields that were holding provider objects in the image heap (and thus initialized at build time). Move all JDK providers (that can be moved) to runtime initialization (see SecurityServicesFeature.java).
  • Allow the loading of security providers via the service loading mechanism (see ServiceLoaderFeature.java).
  • Use a separate structure to support provider verification (see SecurityProvidersSupport), avoiding the need to keep provider objects in the image heap (see SecuritySubstitutions).
  • Allow the instantiation of the security provider at run-time by modifying substitutions in SecuritySubstitutions.
  • Remove the build-time initialization of XMLDSigRI from TrustStoreManager.
  • Update SecurityServiceTest to align with the new changes.

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Nov 22, 2024
@graalvmbot graalvmbot force-pushed the js/GR-57827/providers-in-build-time branch 5 times, most recently from 34af8ac to 6456842 Compare December 3, 2024 09:05
@graalvmbot graalvmbot force-pushed the js/GR-57827/providers-in-build-time branch from 6456842 to 7dcb3a1 Compare December 4, 2024 11:46
@graalvmbot graalvmbot force-pushed the js/GR-57827/providers-in-build-time branch from 7dcb3a1 to 032d899 Compare December 25, 2024 07:14
@graalvmbot graalvmbot force-pushed the js/GR-57827/providers-in-build-time branch 3 times, most recently from 89ff8f9 to 123c4d4 Compare January 15, 2025 08:24
@graalvmbot graalvmbot force-pushed the js/GR-57827/providers-in-build-time branch 2 times, most recently from 7b5e4ab to 945f038 Compare January 23, 2025 13:01
@graalvmbot graalvmbot force-pushed the js/GR-57827/providers-in-build-time branch from 945f038 to d9b4f9d Compare February 19, 2025 09:18
@graalvmbot graalvmbot force-pushed the js/GR-57827/providers-in-build-time branch from d9b4f9d to 8c19084 Compare March 5, 2025 14:27
@graalvmbot graalvmbot force-pushed the js/GR-57827/providers-in-build-time branch from 8c19084 to cabc6c2 Compare March 25, 2025 13:02
@graalvmbot graalvmbot force-pushed the js/GR-57827/providers-in-build-time branch from cabc6c2 to 3c93db2 Compare April 2, 2025 09:54
@graalvmbot graalvmbot force-pushed the js/GR-57827/providers-in-build-time branch from 3c93db2 to 6fa2f31 Compare April 14, 2025 08:32
@graalvmbot graalvmbot force-pushed the js/GR-57827/providers-in-build-time branch from 6fa2f31 to 967efbb Compare May 14, 2025 08:13
@jovanstevanovic
Initialize security providers at run time.
d2b2cef 
Add a test for security provider run time registration.

Add entry to a CHANGELOG.md

Parse java.security.properties file at run time.
@jovanstevanovic
Remove JDK21 code for security provider handling.
a021031
@jovanstevanovic
Remove substitution for Security class for JDK21.
adc30fe
@jovanstevanovic
Update BasedOnJDKFile annotations.
32501d8
@graalvmbot graalvmbot force-pushed the js/GR-57827/providers-in-build-time branch from a96aec3 to 32501d8 Compare May 20, 2025 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@graalvmbot @jovanstevanovic