On Fri, Dec 29, 2017 at 06:56:36PM +0000, Lucas Rolff wrote:
> h2_make_h1_request:153
> h2_frt_decode_headers:2621
> h2_frt_decode_headers:2643
> 
> /* this can be any type of header */
> /* RFC7540#8.1.2: upper case not allowed in header field names */
> for (i = 0; i < list[idx].n.len; i++)
>         if ((uint8_t)(list[idx].n.ptr[i] - 'A') < 'Z' - 'A')
>                 goto fail;
> 
> That's an interesting place to fail

Ah excellent! This also shows a minor bug by which the upper case
'Z' is not correctly caught above. I think I'll add a bit of code
to emit on stderr when in debug mode the header name when this
situation happens. I'll see what can be done, and if we can
consider adding an option to work around this (at least forcing
all names to be lower case).

Thanks,
Willy

Reply via email to