You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Brett Porter (JIRA)" <ji...@codehaus.org> on 2008/08/05 12:07:26 UTC
[jira] Commented: (WAGON-235) LightweightHttpWagon is not using
bulletproof URL.openConnexion(Proxy)
[ http://jira.codehaus.org/browse/WAGON-235?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=144142#action_144142 ]
Brett Porter commented on WAGON-235:
------------------------------------
unfortunately that requires JDK 5.0 - currently the min req is 1.4.2
> LightweightHttpWagon is not using bulletproof URL.openConnexion(Proxy)
> ----------------------------------------------------------------------
>
> Key: WAGON-235
> URL: http://jira.codehaus.org/browse/WAGON-235
> Project: Maven Wagon
> Issue Type: Improvement
> Components: wagon-http-lightweight
> Environment: *
> Reporter: Thomas BELOT
>
> LightweightHttpWagon is messing (even in current trunk r68207) with System.setProperty( "http.proxyHost", proxyInfo.getHost() ); in it's openConnectionInternal method.
> It's clearly not thread safe and a preferred method would be to use URL.openConnexion(Proxy).
> From my point of view, before any call to URL.openConnexion, one should call java.net.ProxySelector#select(URI) and use one (if not every) java.net.Proxy given in the list.
> java.net.Proxy handles care for proxy host and port while java.net.ProxySelector is dealing with nonProxyHost and http/https proxy differenciation
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira