Add FwdState close handler to pinned connections.

Without a close handler, we will not notice when Server forcefully closed the
connection (e.g., when receiving a zero-size response due to a pconn race)
and, hence, will not retry the connection (or server an error), resulting in a
stuck client (e.g., again, when the server received a zero-size response).

diff --git a/src/forward.cc b/src/forward.cc
index 5ed3cc6a9989d21b019e85fe6be25642f19472a9..36c6c798b214c4e4c8462225572fcacd153faeea 100644 (file)
--- a/src/forward.cc
+++ b/src/forward.cc
@@ -913,6 +913,7 @@ FwdState::connectStart()
             request->flags.pinned = 1;
             if (pinned_connection->pinnedAuth())
                 request->flags.auth = 1;
+            comm_add_close_handler(serverConn->fd, fwdServerClosedWrapper, this);
             dispatch();
             return;
         }