RFR: 8341927: Replace hardcoded security providers with new test.provider.name system property [v7]

Sean Mullan mullan at openjdk.org
Wed Oct 30 18:50:05 UTC 2024


On Wed, 30 Oct 2024 18:12:30 GMT, Matthew Donovan <mdonovan at openjdk.org> wrote:

>> In this PR, I removed hard-coded security providers and replaced them with a system property, test.provider.name. If the property is not specified, the provider originally used in the test is used:
>> 
>> Cipher c = Cipher.getInstance("AES/GCM/NoPadding", System.getProperty("test.provider.name", "SunJCE"));
>
> Matthew Donovan has updated the pull request incrementally with one additional commit since the last revision:
> 
>   included additional services in testing.md

Approving but I recommend a follow-on task be filed to check if some of these hard-coded providers are really necessary.

-------------

Marked as reviewed by mullan (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/21551#pullrequestreview-2405973613


More information about the serviceability-dev mailing list