possible fix to issue AsyncHttpClient#114 where if the request times out, we cancel the future (as before); but now also directly make a call to close the channel. Not sure if this impact any pooling on that channel though.

Author: tootedom

providers/netty/src/main/java/com/ning/http/client/providers/netty/NettyResponseFuture.java

@@ -211,7 +211,13 @@ public V get(long l, TimeUnit tu) throws InterruptedException, TimeoutException,
             }
 
             if (expired) {
-                isCancelled.set(true);
+                isCancelled.set(false);
+                try {
+                    channel.getPipeline().getContext(NettyAsyncHttpProvider.class).setAttachment(new NettyAsyncHttpProvider.DiscardEvent());
+                    channel.close();
+                } catch (Throwable t) {
+                    // Ignore
+                }
                 TimeoutException te = new TimeoutException(String.format("No response received after %s %s", l, tu.name().toLowerCase()));
                 if (!throwableCalled.getAndSet(true)) {
                     try {