Header names are not case-sensitive

[marcj]

Header names are not case-sensitive

[WyriHaximus]

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?

[WyriHaximus]

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

[clue]

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

[pwhelan]

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.

[andig]

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

[WyriHaximus]

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

[clue]

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.