[CXF-7883] Handle connectionRequestTimeout in AsyncHTTPConduitFactory

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.2.6
• Fix Version/s: 3.2.8, 3.3.0
• Component/s: Transports
• Labels: pull-request-available
• Estimated Complexity: Unknown
• External issue URL: https://github.com/apache/cxf/pull/466
• Assignee: Freeman Yue Fang
• Reporter: Györgyey Tamás
• Created: 24/Oct/18 15:44
• Updated: 30/Oct/18 18:13
• Resolved: 26/Oct/18 03:18

Description

This issue is a follow-up to CXF-7878.

If connections are contended towards a slow target, it may make sense
to set connectionTimeout and connectionRequestTimeout to values much lower than
the receiveTimeout. Expected client behavior is to receive an error if a connection
does not become available within connectionRequestTimeout. Current behavior however
is that the error is only received after up to receiveTimeout has passed, when a
current request to the target has finished and the connection is released or returned
to the pool.

This causes a possible build-up of pending requests in memory for the duration of receiveTimeout instead of connectionRequestTimeout.

See github PR: https://github.com/apache/cxf/pull/466

The reference solution in the PR works well, but it may not be the most elegant one due to my currently limited understanding of the overall design of the connection handling code.

Issue Links

• links to: GitHub Pull Request #466