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

AsynchHTTPConduit setupConnection - timeout settings

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 3.2.2
    • Transports
    • None
    • 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

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

              Dates

                Created:
                Updated:
                Resolved: