-
-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ChunkedBody decoding with Curl Adapter #25
Comments
It looks like you're leaving off the At any rate, in this case, it looks like the problem is that the server you're talking to sends the header in lowercase: Header names are case-insensitive by definition, and the values for Transfer-Encoding are also case-insensitive, so just adding the Originally posted by @zerocrates at zendframework/zend-http#19 (comment) |
This would be fixed by the open PR #10, and this issue serves as an example of the kind of bug that currently happens and that PR would fix. Specifically, this looks like a regression introduced by ddf5a83: the comparisons were case-sensitive at that point, and when adding support for varying whitespace, the insensitivity was accidentally not applied to the new regex version of the code. Originally posted by @zerocrates at zendframework/zend-http#19 (comment) |
Any idea when this would be fixed? Originally posted by @sgehrig at zendframework/zend-http#19 (comment) |
Nope. I've patched zend to solve this issue. Originally posted by @necromant2005 at zendframework/zend-http#19 (comment) |
Who is the current maintainer of Originally posted by @sgehrig at zendframework/zend-http#19 (comment) |
I suppose that's @weierophinney Originally posted by @necromant2005 at zendframework/zend-http#19 (comment) |
|
Zend\Http\Response::decodeChunkedBody fail with exception when curl adapter decode body itself , BUT it still has "Transfer-Encoding: encoded" header
php: 5.5 , 5.6, 7
Originally posted by @necromant2005 at zendframework/zend-http#19
The text was updated successfully, but these errors were encountered: