[wp-trac] [WordPress Trac] #51056: Fetch_feed parsing of permalinks triggers simplepie preg_match warnings
WordPress Trac
noreply at wordpress.org
Wed Dec 16 00:49:46 UTC 2020
#51056: Fetch_feed parsing of permalinks triggers simplepie preg_match warnings
-------------------------------------------+-----------------------
Reporter: litemotiv | Owner: iandunn
Type: defect (bug) | Status: closed
Priority: high | Milestone: 5.6.1
Component: External Libraries | Version: 5.5
Severity: critical | Resolution: fixed
Keywords: php8 has-patch has-unit-tests | Focuses: rest-api
-------------------------------------------+-----------------------
Changes (by iandunn):
* status: assigned => closed
* resolution: => fixed
Comment:
In [changeset:"49803" 49803]:
{{{
#!CommitTicketReference repository="" revision="49803"
Feed: Merge multiple header values to avoid fatal error.
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.
}}}
--
Ticket URL: <https://core.trac.wordpress.org/ticket/51056#comment:13>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list