[jdk17u-dev] RFR: 8296787: Unify debug printing format of X.509 cert serial numbers

Matthias Baesken mbaesken at openjdk.org
Fri Sep 20 07:29:39 UTC 2024


On Wed, 18 Sep 2024 13:48:59 GMT, Goetz Lindenmaier <goetz at openjdk.org> wrote:

> I backport this for parity with 17.0.14-oracle.
> 
> Patching Copyright failed in most files. Resolved.
> 
> Further conflicts:
> 
> src/java.base/share/classes/sun/security/provider/certpath/Vertex.java
> One conflict due to diffs in the context.
> 
> src/java.base/share/classes/sun/security/ssl/StatusResponseManager.java
> Resolved because code is using append() for Strings instead of '+'.
> 
> test/lib/jdk/test/lib/security/TestTLSHandshake.java
> Resolved due to diffs in context.

Backport looks okay.

There might be still some place(s) where the old output is used.
Is that intentional or was it forgotten to change= But not really related to the backport.
E.g.
src/java.base/share/classes/sun/security/tools/keytool/Main.java
printX509Cert  method
cert.getSerialNumber().toString(16)

-------------

Marked as reviewed by mbaesken (Reviewer).

PR Review: https://git.openjdk.org/jdk17u-dev/pull/2891#pullrequestreview-2317479147


More information about the jdk-updates-dev mailing list