[wp-trac] [WordPress Trac] #25738: WP_HTTP uses transports that incorrectly claim to support a request
WordPress Trac
noreply at wordpress.org
Sat Nov 16 02:19:42 UTC 2013
#25738: WP_HTTP uses transports that incorrectly claim to support a request
--------------------------+------------------
Reporter: dd32 | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: 3.8
Component: HTTP | Version:
Severity: normal | Resolution:
Keywords: |
--------------------------+------------------
Comment (by dd32):
> cURL has a fairly large install base, and the advantage of handling more
in C gives us better performance. Adding parallel request support would
help with this, but I don't think anyone has proposed that yet.
I'm yet to see any significant advantage in performance using cURL, doing
unscientific testing (and excluding any times which were crazy-far from
the rest) showed curl and streams generally within 10ms of each other (on
a 400ms request) on small files, and within 200ms on downloading WordPress
(~5s). When I tried on a slower connection (ironically, my VPS) cURL was
averaging 33s, while Streams averaged 29s.
What I'm saying.. is any significant performance gains between the two are
almost irrelevant for WordPress, if we were making thousands of requests
per hour then squeezing 10ms out here and there makes a huge difference,
but in this case it's more likely remote severs are the performance
bottleneck.
> I'm not sure special-casing it is a good idea; more options = worse.
In general I agree, however, this is a scenario where we need the most
compatibility possible, if HTTPS via cURL fails, we need to try HTTPS via
Streams before falling back to HTTP.
We also don't want to prevent a site from getting update notifications,
which is why we have to have a HTTP fallback.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/25738#comment:13>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list