[wp-trac] Re: [WordPress Trac] #4011: add global proxy support in options

WordPress Trac wp-trac at lists.automattic.com
Wed Mar 4 09:00:22 GMT 2009


#4011: add global proxy support in options
-----------------------------+----------------------------------------------
 Reporter:  tmountjr         |       Owner:  jacobsantos                         
     Type:  feature request  |      Status:  new                                 
 Priority:  normal           |   Milestone:  2.8                                 
Component:  HTTP             |     Version:  2.2                                 
 Severity:  normal           |    Keywords:  proxy has-patch commit needs-testing
-----------------------------+----------------------------------------------
Changes (by DD32):

  * keywords:  proxy has-patch commit => proxy has-patch commit needs-
               testing


Comment:

 > attachment 4011.4.diff added
  * Lots of formatting changes..
  * Fixes a fair few Proxy items
  * Removed the filter 'http_request_port' from the fsockopen transport,
 Seems to not be used anywhere else, Seemed rather pointless there.. The
 port should be set in the URL to start with.
  * Some proxies require the HTTP header to be in the form of "GET
 http://blah/blah/ HTTP/1.0" instead of just utilising "GET /blah HTTP/1.0"
 with a "Host: blah"
  * HTTP Tunneling can cause issues, Many proxies do not allow tunneling
 either (Its used to fetch non-http data, so shouldnt be needed anyway)
  * Better ::test() functions, ::test() can be called from plugins or other
 code now without requiring the 'local'/'ssl' $args to be set(Well.. Why
 not, Else whats the point of the isset()'s?). Should make the ::test()
 functions much more robust now.
  * SSL Verification fixes, use the local ssl verification filter for local
 requests.. Make sure all the transports abide by it
  * Streams transport supports PHP 5.0+, however, Streams + SSL + Proxy
 requires PHP 5.1.0+
  * s/str_replace/preg_replace in certain locations to only replace the
 scheme part, NOT to replace the scheme in other places (ie.
 scheme://somewhere/scheme.php)
  * Passes $args into the filter functions for ::test()
  * A huge patch that needs testing.

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


More information about the wp-trac mailing list