Netty HttpProtocol.finishUpdate should offer channel back to pool after markAsDone, close AsyncHttpClient#844

Author: slandelle

src/main/java/com/ning/http/client/providers/netty/handler/HttpProtocol.java

@@ -181,12 +181,13 @@ private void addType3NTLMAuthorizationHeader(String auth, FluentCaseInsensitiveS
 
     private void finishUpdate(final NettyResponseFuture<?> future, Channel channel, boolean expectOtherChunks) throws IOException {
 
+        markAsDone(future, channel);
+
         boolean keepAlive = future.isKeepAlive();
         if (expectOtherChunks && keepAlive)
             channelManager.drainChannelAndOffer(channel, future);
         else
             channelManager.tryToOfferChannelToPool(channel, keepAlive, future.getPartition());
-        markAsDone(future, channel);
     }
 
     private boolean updateBodyAndInterrupt(NettyResponseFuture<?> future, AsyncHandler<?> handler, NettyResponseBodyPart bodyPart)