Re: [PATCH] http: match headers case-insensitively when redacting

From

Carlo Arenas <carenas@gmail.com>

Date

Sep 21, 2021, 20:14 UTC

Message-ID

<CAPUEspgrqxrhp-5diEenH+vevWi3QtxpjPqTwDuU5J-JHOXg9A@mail.gmail.com>

In-Reply-To

<YUoorS6UwA1DmwBm@coredump.intra.peff.net>

On Tue, Sep 21, 2021 at 12:48 PM Jeff King <peff@peff.net> wrote:

> But certainly I found the behavior surprising. :) I'd guess it's because
> HTTP/2 is sending some binary goo instead of text headers, and the names
> we get are just coming from some lookup table? Or maybe I'm just showing
> my ignorance of HTTP/2.

AFAIK headers in HTTP/2 MUST be lowercase as per the SPEC[1]

Carlo

[1] https://datatracker.ietf.org/doc/html/rfc7540#section-8.1.2