AsyncHttpClient/async-http-client — pull request #169 by tootedom (2012-11-28 10:31:43+00:00)
Title: Possible Fix for issue #114

Hi there,

Hope you are all well.

I'm just wondering if this change to the netty future; cancelling the channel when a request timeout occurs in get(...); addresses the issue with the freeConnections semaphore and the Throttle Filter mentioned in issue #114.

When a request times out in a get(..), should the channel be cancelled like it is when cancel(..) is called on the Netty Future? Locally this looks as though it addresses the issue; but I'm not 100% sure if it has any greater impacts elsewhere (pooling of the channels, etc).

Anyways, see what you think.

Cheers
/dom