DescriptionAllow trailing headers with duplicate keys. Protected by FLAGS_quic_reloadable_flag_quic_allow_duplicate_trailers
QUIC currently treats trailers with duplicate header keys as a critical failure
(causing a CONNECION_CLOSE with QUIC_INVALID_HEADERS_STREAM_DATA). See
b/62770138.
This doesn't seem to be implementing the spec, as RFC 7541 says:
"The dynamic table can contain duplicate entries (i.e., entries with the same
name and same value). Therefore, duplicate entries MUST NOT be treated as an
error by a decoder."
In addition, it doesn't look like our spdy/http2 implementation does any sort
of similar validation check.
This CL makes QUIC more permissive of trailing headers, brings it in line with
our HTTP/2 implementation, and (hopefully) fixes b/62770138.
Merge internal change: 159476633
R=rch@chromium.org
Patch Set 1 #Patch Set 2 : Rebase #
Depends on Patchset: Dependent Patchsets: Messages
Total messages: 10 (8 generated)
|