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