[wp-trac] [WordPress Trac] #25738: WP_HTTP uses transports that incorrectly claim to support a request

WordPress Trac noreply at wordpress.org
Mon Oct 28 02:38:55 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):

 Looking at the chart above with the currently-known bugs, one might
 question why cURL is the first transport we try, it's three fold:
 * cURL was supposably going to have less bugs than our own PHP
 implementation.
  Over the years of WP_HTTP's lifespan we've replaced many functionalities
 of cURL with PHP code (redirects, some error handling, stream
 decompression/decoding) and more recently we've replaced the two PHP-
 implementations (fsockopen and fopen) with a new PHP Streams class (based
 on `stream_context_create()` ) in #25007
 * cURL is more memory efficient as most string operations happen in C,
 which supposably also makes cURL requests faster than Streams. This
 doesn't seem to hold true, as most requests complete in about the same
 time (+/- a %age point)
 * cURL had better SSL verification than Streams, which is no longer true
 as of #25007

 so when it comes down to it, it's mostly a legacy ordering, but there
 exists code out there which expects HTTP requests to be made via cURL and
 doesn't take into account its possible for it to use something else.

--
Ticket URL: <http://core.trac.wordpress.org/ticket/25738#comment:2>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list