Skip to content
New issue

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

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

Already on GitHub? # to your account

http3: add http3.Server.ServeQUICConn to serve a single QUIC connection #3579

Merged
merged 1 commit into from
Oct 11, 2022

Conversation

marten-seemann
Copy link
Member

@marten-seemann marten-seemann commented Oct 9, 2022

This is needed when having a QUIC listener that accepts multiple ALPNs. If the HTTP/3 ALPN is negotiated, one would pass the connection to http3.Server.ServerQUICConn. If a different ALPN is negotiated, the connection would be passed to a different connection handler.

@marten-seemann marten-seemann added this to the v0.30.0 milestone Oct 9, 2022
@codecov
Copy link

codecov bot commented Oct 9, 2022

Codecov Report

Base: 85.50% // Head: 85.49% // Decreases project coverage by -0.02% ⚠️

Coverage data is based on head (7d6b0fc) compared to base (424a663).
Patch coverage: 53.85% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3579      +/-   ##
==========================================
- Coverage   85.50%   85.49%   -0.02%     
==========================================
  Files         139      139              
  Lines       10244    10252       +8     
==========================================
+ Hits         8759     8764       +5     
- Misses       1103     1105       +2     
- Partials      382      383       +1     
Impacted Files Coverage Δ
http3/server.go 74.01% <53.85%> (-0.29%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants