add global proxy support in options
|Reported by:||tmountjr||Owned by:||jacobsantos|
|Severity:||normal||Keywords:||proxy has-patch needs-testing|
I use the 2.2-bleeding SVN build of WP on a local install to test upcoming changes to the system before putting them on my live blog. In order to test accurately, I installed FeedWordPress to pull my live RSS feed. I couldn't figure out why it wouldn't pull my feeds (even after adding all the appropriate snoopy proxy variables in the plugin) unless I set the proxy in wp-includes/class-snoopy.php. Ticket 3082 details the same problem. Perhaps a set of define statements could be added to wp-config.php:
In wp-includes/class-snoopy.php, for instance, the current code could be changed to respect the proxy settings if entered:
var $proxy_host = WP_PROXYHOST;
var $proxy_port = WP_PROXYPORT;
var $_isproxy = (WP_USEPROXY == 'TRUE') : true ? false;
These settings could also be carried over to Akismet and any other class that uses snoopy or fsockopen; I successfully tested this in class-snoopy.php. Besides not remembering my ternary operators for the _isproxy variable, the change works and applies to all instances where WP is using snoopy to get something from the web.
I'd be happy to work up something official if anyone thinks it's worth it...
Change History (51)
- Keywords snoopy needs-testing removed
- Type changed from enhancement to feature request
- Keywords commit added; needs-testing removed
- Milestone changed from 2.9 to 2.8