On Wed, 3 Jun 2026 14:22:05 GMT, Sean Mullan <[email protected]> wrote:

> This change adds the negotiated named group to the JFR TLSHandshakeEvent.
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

src/java.base/share/classes/sun/security/ssl/Finished.java line 1191:

> 1189:             }
> 1190:             long peerCertificateId = Integer.toUnsignedLong(hash);
> 1191:             String namedGroup = "N/A";

Not sure if it matters, but more expensive operations are typically moved 
inside shouldCommit() so they are only executed if the event is about to be 
committed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/31371#discussion_r3361444806

Reply via email to