[wp-trac] [WordPress Trac] #33055: Support Parallel HTTP Requests in WP_Http, et al
WordPress Trac
noreply at wordpress.org
Sun Apr 10 19:31:22 UTC 2016
#33055: Support Parallel HTTP Requests in WP_Http, et al
---------------------------------+-----------------------------
Reporter: wonderboymusic | Owner: rmccue
Type: task (blessed) | Status: assigned
Priority: normal | Milestone: Future Release
Component: HTTP API | Version:
Severity: normal | Resolution:
Keywords: early needs-refresh | Focuses: performance
---------------------------------+-----------------------------
Comment (by ocean90):
Some random thoughts about [attachment:33055.2.diff]:
* Requests uses `spl_autoload_register()` so we have to provide a fallback
in case SPL is disabled.
* `WP_Requests_Hooks` needs some whitespace love. Is it part of Request or
WP? Asking because of the `@package` tag and because it's in the root of
`/wp-includes/`.
* `/wp-includes/requests/Requests/` looks weird. Can we move it up one
dir?
* What's the status of https://github.com/rmccue/Requests-WPHTTP/issues/1?
* Are they any other tickets which could be marked as fixed as soon as
Requests lands?
(https://core.trac.wordpress.org/query?status=!closed&component=HTTP+API)
* Can we get a refresh of the patch so it could potentially land on day
one of 4.6?
* "We can also remove the HTTP tests from the core tests if you'd like,
since they're all covered by Requests' tests already. " I wouldn't remove
them directly. Let's make sure that our tests are passing too and remove
them only when the implementation is finished.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/33055#comment:19>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list