[wp-trac] [WordPress Trac] #53938: replace core uses of wp_parse_url() with PHP's native parse_url()
WordPress Trac
noreply at wordpress.org
Tue Aug 17 00:55:14 UTC 2021
#53938: replace core uses of wp_parse_url() with PHP's native parse_url()
-------------------------------------------+------------------------------
Reporter: pbiron | Owner: (none)
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting Review
Component: HTTP API | Version:
Severity: normal | Resolution:
Keywords: needs-patch 2nd-opinion close | Focuses:
-------------------------------------------+------------------------------
Comment (by jrf):
@pbiron While I would very much welcome more tests, in this case, the
tests are only relevant for those calls which ''may'' run into the `:`
issue. So I suspect that would be the calls to `parse_url()` without the
`$component` parameter or when the `$component` would be any of these:
`PHP_URL_QUERY`, `PHP_URL_PORT`, but I haven't looked into the colon issue
in detail today to bring back the fine details, so the above suggestion
should be verified carefully against the PHP core ticket:
https://bugs.php.net/bug.php?id=74780
--
Ticket URL: <https://core.trac.wordpress.org/ticket/53938#comment:8>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list