You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cxf.apache.org by "Antonin Fischer (JIRA)" <ji...@apache.org> on 2018/01/25 13:59:00 UTC
[jira] [Updated] (CXF-7623) AsynchHTTPConduit setupConnection -
timeout settings
[ https://issues.apache.org/jira/browse/CXF-7623?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Antonin Fischer updated CXF-7623:
---------------------------------
Description:
Class AsynchHTTPConduit method setupConnection(). Codeblock:
{code:java}
RequestConfig.Builder b = RequestConfig.custom()
.setConnectTimeout((int) csPolicy.getConnectionTimeout())
.setSocketTimeout((int) csPolicy.getReceiveTimeout())
.setConnectionRequestTimeout((int) csPolicy.getReceiveTimeout());
{code}
Line *setConnectionRequestTimeout((int) csPolicy.getReceiveTimeout())*
Settings of request timeout with receive timeout is not backward compatible solution. In addition the receive timeout is in many times to long. Better way is set the *request timeout by connection timeout* instead of it. It is still compromis solution.
HTTPClientPolicy object still doesn't contain request timeout. Think about extension of domain objects about request timeout which is already used in pools etc.
was:
Class AsynchHTTPConduit method setupConnection().
Code part:
{code:java}
RequestConfig.Builder b = RequestConfig.custom()
.setConnectTimeout((int) csPolicy.getConnectionTimeout())
.setSocketTimeout((int) csPolicy.getReceiveTimeout())
.setConnectionRequestTimeout((int) csPolicy.getReceiveTimeout());
{code}
Line *setConnectionRequestTimeout((int) csPolicy.getReceiveTimeout())*
Settings of request timeout with receive timeout is not backward compatible solution. In addition the receive timeout is in many times to long. Better way is set the *request timeout by connection timeout* instead of it. It is still compromis solution.
HTTPClientPolicy object still doesn't contain request timeout. Think about extension of domain objects about request timeout which is already used in pools etc.
> AsynchHTTPConduit setupConnection - timeout settings
> ----------------------------------------------------
>
> Key: CXF-7623
> URL: https://issues.apache.org/jira/browse/CXF-7623
> Project: CXF
> Issue Type: Bug
> Components: Transports
> Reporter: Antonin Fischer
> Priority: Minor
>
> Class AsynchHTTPConduit method setupConnection(). Codeblock:
> {code:java}
> RequestConfig.Builder b = RequestConfig.custom()
> .setConnectTimeout((int) csPolicy.getConnectionTimeout())
> .setSocketTimeout((int) csPolicy.getReceiveTimeout())
> .setConnectionRequestTimeout((int) csPolicy.getReceiveTimeout());
> {code}
> Line *setConnectionRequestTimeout((int) csPolicy.getReceiveTimeout())*
> Settings of request timeout with receive timeout is not backward compatible solution. In addition the receive timeout is in many times to long. Better way is set the *request timeout by connection timeout* instead of it. It is still compromis solution.
> HTTPClientPolicy object still doesn't contain request timeout. Think about extension of domain objects about request timeout which is already used in pools etc.
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)