Do not upper case first character of headers in socket adapter#161
Do not upper case first character of headers in socket adapter#161DaDeather wants to merge 1 commit intozendframework:masterfrom DaDeather:master
Conversation
|
May be make this feature optional, like property in class by default use ucfirst and if you need you could change that behaviour?) |
I can't tell if it would make sense to have a property to configure whatever it should be upper cased or not. The only thing I can say is that its already not being modified for the Curl and Proxy adapter. |
|
That make sense i think, i was just thinking about back compatibility, but that make no sense in that case) also some test are not successful after your changes, i think you need investigate test report) |
Do not upper case first character of headers in socket adapter
|
Thanks, @DaDeather; merged to develop for release with 2.9.0. |
This PR Fixes #160 which causes headers being upper cased for their first character in key which causes a fail request to a non HTTP 1.1 compliant API.
HTTP 1.1 defines that the headers should be case insensitive. This means there is also no reason for upper casing the first character.
The first character in a headers "name"/"key" was being upper cased in default for Socket and Test adapter.
The headers using the Socket and Test adapter should not be be upper cased for the first character any more.