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