Skip to content

tls: report negotiated TLS groups - #64119

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
panva:tls-report-negotiated-groups
Jul 1, 2026
Merged

tls: report negotiated TLS groups#64119
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
panva:tls-report-negotiated-groups

Conversation

@panva

@panva panva commented Jun 24, 2026

Copy link
Copy Markdown
Member

tlsSocket.getEphemeralKeyInfo() currently returns an empty object for TLS 1.3 key agreement groups where OpenSSL does not expose a peer temporary key object, including ML-KEM and hybrid ML-KEM groups.

This updates the shared TLS helper to fall back to OpenSSL's negotiated TLS group name and return { type: 'TLSGroup', name: '<group>' }.

The existing DH and ECDH result shapes are unchanged and still take precedence when OpenSSL provides a recognizable temporary key. TLSGroup does not include size, because Node cannot derive a meaningful single key size for PQ or hybrid groups.

The TLS docs now describe TLSGroup, point users to the IANA TLS Supported Groups registry, and clarify that the historical ecdhCurve option also configures TLS Supported Groups for TLS 1.3.

Fixes: #59452

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs with CI started, the required approvals, and no outstanding review comments. lib / src Issues and PRs involving general changes in the lib/ or src/ directories. needs-ci PRs that need a full CI run. review wanted PRs that need review. semver-minor PRs that contain new features and should be released in the next minor version. tls Issues and PRs related to the tls subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tlsSocket.getEphemeralKeyInfo() - doesn't display information for hybrid groups

4 participants