HTTP/2 headers should all be lowercase

[jwoertink]

According to the HTTP/2 spec, all headers should be lowercase.

Currently crystal handles checking against lowercase vs uppercase, and dashes vs underscores, so CONTENT_TYPE is the same as content-type. We could safely lowercase, and dasherize for convention any place that we mention a header key directly.

This also makes me curious to know what else we would need to do to ensure we fully support HTTP/2? 🤔

[jkthorne]

should this be a patch to the Crystal project?

[jwoertink]

Yeah, it probably should be. Does the HTTP::Server even support HTTP/2 yet?

[jkthorne]

I do not believe so.
but there is a reasonably well implementation here https://github.com/ysbaddaden/http2

[jwoertink]

With servers like Caddy enabling HTTP/3 .... I wonder if there's stuff we would need to consider 🤔 I know less than Jon Snow about it all