Feed: Merge multiple header values to avoid fatal error.

Browse Source

When SimplePie parses HTTP headers, it combines multiple values for the same header into a comma-separated string. `WP_SimplePie_File` overrides the parsing, but was leaving them as an array instead.

That lead to a fatal error in PHP 8, because other parts of the codebase ended up passing an array to a function that expected a string.

Props david.binda, litemotiv, inc2734, NicolasKulka, hellofromTonya, mbabker, skithund, SergeyBiryukov, desrosj, timothyblynjacobs.
Fixes #51056. See #51956.



git-svn-id: https://develop.svn.wordpress.org/trunk@49803 602fd350-edb4-49c9-b593-d223f7449a82

...

This commit is contained in:

Ian Dunn

2020-12-16 00:49:32 +00:00

parent 378c2058a0

commit 4595dcf7f9

3 changed files with 1059 additions and 1 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download Patch File Download Diff File Expand all files Collapse all files

910

tests/phpunit/data/feed/wordpress-org-news.xml Normal file

File diff suppressed because one or more lines are too long