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

Author: slandelle

providers/netty4/src/main/java/org/asynchttpclient/providers/netty4/handler/HttpProtocol.java

@@ -190,12 +190,13 @@ private void addType3NTLMAuthorizationHeader(String authenticateHeader, FluentCa
 
     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.getPartitionId());
-        markAsDone(future, channel);
     }
 
     private boolean updateBodyAndInterrupt(NettyResponseFuture<?> future, AsyncHandler<?> handler, NettyResponseBodyPart bodyPart) throws Exception {