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

Jaroslav Bielcik updated HTTPCLIENT-806:
----------------------------------------

    Component/s: HttpClient

> ConnectException not handled in DefaultHttpMethodRetryHandler
> -------------------------------------------------------------
>
>                 Key: HTTPCLIENT-806
>                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-806
>             Project: HttpComponents HttpClient
>          Issue Type: Bug
>          Components: HttpClient
>    Affects Versions: 3.1 Final
>            Reporter: Jaroslav Bielcik
>             Fix For: 4.0.1
>
>   Original Estimate: 0.25h
>  Remaining Estimate: 0.25h
>
> Copied from my mailing list post, Oleg suggested I post it to JIRA for 4.0 
> fix.
> i am using commons-httpclient.3.0.1 and I am sending some requests
> through https protocol. I have a problem with a long creation of
> connection if ip address of remote service is not existing. I think
> problem is in the situation when https connection is not created and
> ConnectException is thrown after connection timeout. This exception is
> catched in HttpMethodDirector.java in method executeWithRetry. Then
> the DefaultHttpMethodRetryHandler is called to recognize whether
> connection creation will be repeated or not.
> I think, that special handling for ConnectException is missing in
> retryMethod of DefaultHttpMethodRetryHandler, because exception is not
> recognized and connetions are created again.
> On the other hand, ConnectTimeoutException is thrown after connection
> timeout for HTTP. This exception is handled in
> DefaultHttpMethodRetryHandler and call is stopped.
> These lines of code handle ConnectTimeoutException in retryMethod of
> DefaultHttpMethodRetryHandler:
> if (exception instanceof InterruptedIOException) {
>             // Timeout
>             return false;
>         }
> Probably this is missing for ConnectException:
> if (exception instanceof InterruptedIOException || exception
> instanceof ConnectException) {
>             // Timeout
>             return false;
>         }

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to