Integrated: 8327779: Remove deprecated internal field sun.security.x509.X509Key.key
Eirik Bjørsnøs
eirbjo at openjdk.org
Thu Mar 28 20:02:35 UTC 2024
On Mon, 11 Mar 2024 09:58:42 GMT, Eirik Bjørsnøs <eirbjo at openjdk.org> wrote:
> Please review this cleanup PR which removes the protected, deprecated field `X509Key.key` in the internal package `sun.security.x509`.
>
> This field and the associated field `unusedBits` have been marked `@Deprecated` since the initial load. The recommended replacement is to use the `BitArray` representation, which does not require keys to be byte-aligned.
>
> Two use sites of the `key` field were found in subclasses:
>
> * The `ECPublicKeyImpl` constructor was updated to use a `BitArray` instead.
> * `ECPublicKeyImpl.parseKeyBits()` was updated to call `getKey().toByteArray()` to produce the byte array.
>
> Somewhat unrelated, the method `ECPublicKeyImpl.getEncodedPublicValue()` was deemed unused and has been removed.
>
> Testing and verification: This PR does not update any tests. GHA runs green.
This pull request has now been integrated.
Changeset: 4eefda91
Author: Eirik Bjørsnøs <eirbjo at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/4eefda91021e07768ace03725c88a56af33ca344
Stats: 55 lines in 2 files changed: 2 ins; 48 del; 5 mod
8327779: Remove deprecated internal field sun.security.x509.X509Key.key
Reviewed-by: mullan
-------------
PR: https://git.openjdk.org/jdk/pull/18185
More information about the security-dev
mailing list