Uploaded image for project: 'CXF'
  1. CXF
  2. CXF-7623

AsynchHTTPConduit setupConnection - timeout settings

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.2.2
    • Component/s: Transports
    • Labels:
      None
    • Estimated Complexity:
      Unknown

      Description

      Class AsynchHTTPConduit method setupConnection(). Codeblock:

      RequestConfig.Builder b = RequestConfig.custom()
                      .setConnectTimeout((int) csPolicy.getConnectionTimeout())
                      .setSocketTimeout((int) csPolicy.getReceiveTimeout())
                      .setConnectionRequestTimeout((int) csPolicy.getReceiveTimeout());
      

      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.

       

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                ffang Freeman Yue Fang
                Reporter:
                raven2cz Antonin Fischer
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: