Skip to content
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

Zend_Http_Client-Adapter_Socket - check transfer-encoding header is a string (and not an array) #396

Merged
merged 3 commits into from
Jan 26, 2024

Conversation

DavidGoodwin
Copy link

I saw this error reported in some server logs -

PHP Fatal error: Uncaught TypeError: strtolower(): Argument #1 ($string) must be of type string, array given in ... /Zend/Http/Client/Adapter/Socket.php:362

Most http headers can be a single stringy value or an array of strings ... I'm not sure what the correct behaviour should be if there is more than one transfer-encoding (it feels like it's an error on the server my code connected to?).

Anyway, after this, it'll just throw a Zend_Http_Client_Adapter_Exception instead - which feels like the only option?

…($string) must be of type string, array given in ... /Zend/Http/Client/Adapter/Socket.php:362
@develart-projects develart-projects merged commit dbdd481 into Shardj:master Jan 26, 2024
8 checks passed
@develart-projects develart-projects added this to the 1.24.0 milestone Jan 26, 2024
@develart-projects develart-projects added the to be released PR exists or in master, but not released yet label Jan 26, 2024
@DavidGoodwin
Copy link
Author

Thanks for merging!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
to be released PR exists or in master, but not released yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants