On Fri, 25 Oct 2024 17:15:34 GMT, Matthew Donovan <[email protected]> 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 with a new target base due to a
> merge or a rebase. The incremental webrev excludes the unrelated changes
> brought in by the merge/rebase. The pull request contains five additional
> commits since the last revision:
>
> - removed a couple instances of provider property
> - Merge branch 'master' into jce-provider-prop
> - fixed whitespace
> - Updated a few more tests.
> - 8341927: Remove hardcoded SunJCE provider
Marked as reviewed by ascarpino (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/21551#pullrequestreview-2403191902
