[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