Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

QUIC, 0RTT for gnutls #15667

Closed
wants to merge 2 commits into from
Closed

QUIC, 0RTT for gnutls #15667

wants to merge 2 commits into from

Conversation

icing
Copy link
Contributor

@icing icing commented Dec 2, 2024

When a QUIC TLS session announced early data support and 'CURLSSLOPT_EARLYDATA' is set for the transfer, send initial request and body (up to the 128k we buffer) as 0RTT when curl is built with ngtcp2+gnutls.

QUIC 0RTT needs not only the TLS session but the QUIC transport paramters as well. Store those and the earlydata max value together with the session in the cache.

Add test case for h3 use of this. Enable quic early data in nghttpx for testing.

Update: it's probably easiest to merge #15774 first and then rebase this one on top.

@github-actions github-actions bot added the tests label Dec 2, 2024
@icing icing added HTTP/3 h3 or quic related feature-window A merge of this requires an open feature window labels Dec 2, 2024
@bagder
Copy link
Member

bagder commented Dec 21, 2024

After fa0ccd9 landed this needs some attention...

When a QUIC TLS session announced early data support and
'CURLSSLOPT_EARLYDATA' is set for the transfer, send initial
request and body (up to the 128k we buffer) as 0RTT when curl
is built with ngtcp2+gnutls.

QUIC 0RTT needs not only the TLS session but the QUIC transport
paramters as well. Store those and the earlydata max value
together with the session in the cache.

Add test case for h3 use of this. Enable quic early data in
nghttpx for testing.
@icing icing requested a review from bagder December 23, 2024 13:24
@bagder bagder closed this in 68bd759 Dec 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-window A merge of this requires an open feature window HTTP/3 h3 or quic related tests
Development

Successfully merging this pull request may close these issues.

2 participants