On Fri, 8 Dec 2023 19:07:27 GMT, Ben Perez <d...@openjdk.org> wrote: >> Updated `X509CertSelector.java`, `SerialNumber.java`, and `SSLLogger` to >> use same debug printing format for X.509 serial numbers. The changes are: >> - `X509CertSelector` was `SN: 83be056904246b1a1756ac95991c74a` and now is >> `Serial number: 08:3B:E0:56:90:42:46:B1:A1:75:6A:C9:59:91:C7:4A` >> - `SerialNumber` was `SerialNumber: [ 083be056 904246b1 a1756ac9 5991c74a]` >> and now is `Serial number: 08:3B:E0:56:90:42:46:B1:A1:75:6A:C9:59:91:C7:4A` >> - `SSLLogger` was `"serial number" : "083BE056904246B1A1756AC95991C74A"` and >> now is `"serial number" : 08:3B:E0:56:90:42:46:B1:A1:75:6A:C9:59:91:C7:4A` > > Ben Perez has updated the pull request incrementally with one additional > commit since the last revision: > > added format check to X509CertSelectorTest
Looks good to me. Please ensure you run a wide selection of tests before integration. It should include JFR testing to exercise the X509 JFR event testing we have there. ------------- Marked as reviewed by coffeys (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/16834#pullrequestreview-1775475863
