hc-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Oleg Kalnichevski (JIRA)" <j...@apache.org>
Subject [jira] Resolved: (HTTPCLIENT-1058) SO_TIMEOUT not set early enough for SOCKS proxies in PlainSocketFactory
Date Tue, 15 Feb 2011 15:54:57 GMT

     [ https://issues.apache.org/jira/browse/HTTPCLIENT-1058?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Oleg Kalnichevski resolved HTTPCLIENT-1058.
-------------------------------------------

       Resolution: Fixed
    Fix Version/s: 4.2 Final

SOCKS is a TCP level, not an HTTP level protocol. If configured correctly, it should be completely
transparent to HttpClient. However, I am open to contributions of a better, more integrated
support for SOCKS.

I committed the proposed fix to SVN trunk.

Oleg

 

> SO_TIMEOUT not set early enough for SOCKS proxies in PlainSocketFactory
> -----------------------------------------------------------------------
>
>                 Key: HTTPCLIENT-1058
>                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-1058
>             Project: HttpComponents HttpClient
>          Issue Type: Bug
>          Components: HttpConn
>    Affects Versions: 4.1 Final
>            Reporter: subes
>            Priority: Minor
>             Fix For: 4.2 Final
>
>
> I've created my own delegating SchemeSocketFactory implementation which supports setting
SOCKS proxies on socket creation. In the connectSocket implementation, I previously just delegated
to PlainSocketFactory.
> The problem there was, that the SO_TIMEOUT was not set on the socket before the connection
was established through the SOCKS proxy. This lead to a stop on the native read0 method because
the socket is endlessly waiting for a read to occur from the proxy, so it can continue with
the the connect to the actual socket destination through the proxy. I made sure I set the
SO_TIMEOUT parameter in HttpParams, but it did not get honored by PlainSocketFactory.
> To fix this and make HttpClient honor SO_TIMEOUT for SOCKS proxies, the following line
has to be added:
>   sock.setSoTimeout(HttpConnectionParams.getSoTimeout(params));
> in PlainSocketFactory.connectSocket(...).
> Heres the complete fixed method:
> PlainSocketFactory:            
>     public Socket connectSocket(
>             final Socket socket,
>             final InetSocketAddress remoteAddress,
>             final InetSocketAddress localAddress,
>             final HttpParams params) throws IOException, ConnectTimeoutException {
>         if (remoteAddress == null) {
>             throw new IllegalArgumentException("Remote address may not be null");
>         }
>         if (params == null) {
>             throw new IllegalArgumentException("HTTP parameters may not be null");
>         }
>         Socket sock = socket;
>         if (sock == null) {
>             sock = createSocket();
>         }
>         if (localAddress != null) {
>             sock.setReuseAddress(HttpConnectionParams.getSoReuseaddr(params));
>             sock.bind(localAddress);
>         }
>         
>         //FIX for SOCKS proxies which get stalled if they don't answer
>         sock.setSoTimeout(HttpConnectionParams.getSoTimeout(params));
>         
>         int timeout = HttpConnectionParams.getConnectionTimeout(params);
>         try {
>             sock.connect(remoteAddress, timeout);
>         } catch (SocketTimeoutException ex) {
>             throw new ConnectTimeoutException("Connect to " + remoteAddress.getHostName()
+ "/"
>                     + remoteAddress.getAddress() + " timed out");
>         }
>         return sock;
>     }
> Currently I've implemented this in my delegating SchemeSocketFactory, because PlainSocketFactory
misses this setting.
> Dunno if there are other implementations of SocketFactory in HttpClient, which might
need this fix. Anyway I hope this helps other people who get  headaches about halting threads
because they use SOCKS proxies. :)

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
For additional commands, e-mail: dev-help@hc.apache.org


Mime
View raw message