try detecting outbound-complete with autoRead only if non keep alive

leave onOutboundComplete takes precedence for terminal event
clear channelOperation before writing response

Author: Stephane Maldini

src/main/java/reactor/ipc/netty/http/server/HttpServerOperations.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2011-2018 Pivotal Software Inc, All Rights Reserved.
+ * Copyright (c) 2011-2017 Pivotal Software Inc, All Rights Reserved.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -367,13 +367,9 @@ protected void onInboundNext(ChannelHandlerContext ctx, Object msg) {
 			}
 			if (msg instanceof LastHttpContent) {
 				onInboundComplete();
-				if (isOutboundDone()) {
-					onHandlerTerminate();
-				}
-				else {
-					//force auto read to enable more accurate close selection now inbound is done
+				if (!isKeepAlive()) {
 					channel().config()
-					         .setAutoRead(true);
+					         .setAutoRead(true); //detect close
 				}
 			}
 		}
@@ -388,6 +384,8 @@ protected void onOutboundComplete() {
 			return;
 		}
 
+		onHandlerTerminate();
+
 		final ChannelFuture f;
 		if (log.isDebugEnabled()) {
 			log.debug("Last HTTP response frame");
@@ -404,16 +402,15 @@ else if (markSentBody()) {
 		}
 		else{
 			discard();
-			onHandlerTerminate();
 			return;
 		}
 		f.addListener(s -> {
 			discard();
-			onHandlerTerminate();
 			if (!s.isSuccess() && log.isDebugEnabled()) {
-				log.error("Failed flushing last frame", s.cause());
+				log.error(channel()+" Failed flushing last frame", s.cause());
 			}
 		});
+
 	}
 
 	@Override