RFR: 8325162: Remove duplicate GCMParameters class [v2]
Daniel Jeliński
djelinski at openjdk.org
Thu Feb 15 11:29:58 UTC 2024
On Tue, 13 Feb 2024 20:58:17 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:
>> Currently we have 2 GCMParameters classes in java.base, one in sun.security.util package, the other in com.sun.crypto.provider.
>>
>> This PR merges the changes from the sun.security.util copy into com.sun.crypto.provider, and removes the sun.security.util class.
>>
>> As far as I could tell, the sun.security.util class was added to avoid exporting com.sun.crypto.provider to jdk.crypto.cryptoki. The export was introduced later in [JDK-8259319](https://bugs.openjdk.org/browse/JDK-8259319), so there's no reason to keep both copies.
>>
>> No new tests. Tier1-3 tests continue to pass.
>
> Daniel Jeliński has updated the pull request incrementally with one additional commit since the last revision:
>
> Remove the other GCMParameters
Thanks for the discussion.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/17802#issuecomment-1945903287
More information about the security-dev
mailing list