BUG/MEDIUM: qpack: allow 6xx..9xx status codes

HTTP status codes outside of 100..599 are considered invalid in HTTP
RFC9110. However, it is explicitely stated that range 600..999 is often
used for internal communication so in practice haproxy must be lenient
with it.

Before this patch, QPACK encoder rejected these values. This resulted in
a connection error. Fix this by extending the range of allowed values
from 100 to 999.

This is linked to github issue #2422. Once again, thanks to @yokim-git
for his help here.

This must be backported up to 2.6.
This commit is contained in:
Amaury Denoyelle 2024-01-29 13:45:48 +01:00
parent 5d2fe1871a
commit 7d22c4956c

View File

@ -69,7 +69,7 @@ int qpack_encode_int_status(struct buffer *out, unsigned int status)
{
int status_size, idx = 0;
if (status < 100 || status > 599)
if (status < 100 || status > 999)
return 1;
switch (status) {