[ 
https://issues.apache.org/jira/browse/HTTPCLIENT-1765?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16911983#comment-16911983
 ] 

Jang Ryeol commented on HTTPCLIENT-1765:
----------------------------------------

Why is this marked invalid? connectTimeout should not be used as socketTimeout.

[https://github.com/apache/httpcomponents-client/commit/d954cd287dfcdad8f153e61181e20d253175ca8c]

>From the above commit, connectTimeout also works as readTimeout when no 
>readTimeout specified.

HttpComponentsClientHttpRequestFactory clientHttpRequestFactory = new 
HttpComponentsClientHttpRequestFactory();
clientHttpRequestFactory.setConnectTimeout(1000);
restTemplate = new RestTemplate(clientHttpRequestFactory);

So when you use restTemplate setup above, ReadTimeoutException occurs when 
requests take more than 1s.

> SSLConnectionSocketFactory uses connectTimeout for read timeout
> ---------------------------------------------------------------
>
>                 Key: HTTPCLIENT-1765
>                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-1765
>             Project: HttpComponents HttpClient
>          Issue Type: Bug
>          Components: HttpClient (classic)
>    Affects Versions: 4.5
>         Environment: Any
>            Reporter: SATISH BURNWAL
>            Priority: Major
>              Labels: features
>
> SSLConnectionSocketFactory uses connect timeout value for socket.soTimeout as 
> well (as per the code in SSLConnectionsocketFactory). Because of this, when 
> clients are created with such config (below), read timeout is not taking 
> effect.
> RequestConfig.Builder rb = RequestConfig.custom();
>                       rb.setConnectTimeout(3000);
>                       rb.setExpectContinueEnabled(true);
>                       rb.setSocketTimeout(10000);
>                       rb.setAuthenticationEnabled(true);



--
This message was sent by Atlassian Jira
(v8.3.2#803003)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to