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/jdk/internal/event/TLSHandshakeEvent.java line 38:

> 36:     public String cipherSuite;
> 37:     public long certificateId;
> 38:     public String namedGroup;

Nit: `namedGroup` is being logged right after `cipherSuite`, it will be 
consistent to keep the same order here and in 
[`src/jdk.jfr/share/classes/jdk/jfr/events/TLSHandshakeEvent.java`](https://github.com/openjdk/jdk/pull/31371/changes#diff-6fcb6ccf38f77d1a943ad82829a7b70a3d28ace92e895897963a474222675ed4)

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

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

Reply via email to