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

Http/3 :status static response compression missing values #38604

Closed
Tratcher opened this issue Nov 23, 2021 · 0 comments · Fixed by #38605
Closed

Http/3 :status static response compression missing values #38604

Tratcher opened this issue Nov 23, 2021 · 0 comments · Fixed by #38605
Labels
area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions feature-kestrel HTTP3

Comments

@Tratcher
Copy link
Member

switch (statusCode)
{
case 200:
case 204:
case 206:
case 304:
case 400:
case 404:
case 500:
QPackEncoder.EncodeStaticIndexedHeaderField(H3StaticTable.StatusIndex[statusCode], buffer, out var bytesWritten);

This list was copied from HTTP/2, but HTTP/3's qpack static table contains more entries:
https://quicwg.org/base-drafts/draft-ietf-quic-qpack.html#name-static-table-2
24 :status 103
25 :status 200
26 :status 304
27 :status 404
28 :status 503
63 :status 100
64 :status 204
65 :status 206
66 :status 302
67 :status 400
68 :status 403
69 :status 421
70 :status 425
71 :status 500

@JamesNK

@ghost ghost locked as resolved and limited conversation to collaborators Dec 26, 2021
@amcasey amcasey added area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions and removed area-runtime labels Jun 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions feature-kestrel HTTP3
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants