Ensure the conditions to log "accepting..." match those for "closing..."

For instance, the following test led to "accepting AMQP connection"
being logged, but no corresponding "closing AMQP connection":
  $ nc localhost 5672
  a
  ^C

The same issue applied to TLS connections.

References bug26427.

1 files changed, 3 insertions, 1 deletions

diff --git a/src/rabbit_reader.erl b/src/rabbit_reader.erl
index 1cc12dbd16..94723fbae1 100644
--- a/src/rabbit_reader.erl
+++ b/src/rabbit_reader.erl
@@ -350,6 +350,8 @@ mainloop(Deb, Buf, BufLen, State = #v1{sock = Sock,
                      State#v1{pending_recv = false});
         closed when State#v1.connection_state =:= closed ->
             ok;
+        closed when CS =:= pre_init andalso Buf =:= [] ->
+            stop(tcp_healthcheck, State);
         closed ->
             stop(closed, State);
         {error, Reason} ->
@@ -364,7 +366,7 @@ mainloop(Deb, Buf, BufLen, State = #v1{sock = Sock,
             end
     end.
 
-stop(closed, #v1{connection_state = pre_init} = State) ->
+stop(tcp_healthcheck, State) ->
     %% The connection was closed before any packet was received. It's
     %% probably a load-balancer healthcheck: don't consider this a
     %% failure.