Header names are not case-sensitive about http HOT 7 CLOSED

Thanks for reporting and you're absolutely right, this has been on my mind for a while. I'm in favor of strtolower all the header names.

@clue what is your take on this?

from http.

Or could go for a class that behaves as an array and makes them case-insensitive

from http.

but I believe we should fix it directly in reactphp/http

Absolutely! 👍

Afaict most occurrences will be replaced with PR #41 anyway, but we should definitely keep an eye on this.

I'm in favor of strtolower all the header names.

Yeah, this should do it for now (http://php.net/array_change_key_case).

Eventually we will look into supporting PSR-7 (#28), by then we will also follow proper message semantics while preserving header case information.

FWIW, RFC 2616 has been replaced with this:

Each header field consists of a case-insensitive field name followed by […]
http://tools.ietf.org/html/rfc7230#section-3.2

from http.

I actually went ahead and implemented a fix in PR #53 since I needed a fix for a work project. Hopefully #41 gets merged soon so I can drop my fix.

from http.

Will we ever see #41? It's a year in making now (sad).

from http.

@andig It's nearing completion see #41 (comment) see the issues mentioned in that comment. You're input on those is valued 👍

from http.

I'm currently looking into this and will file a PR for the upcoming v0.4.4 release. See #103 for the first step here.

from http.