Set logged status code (%<Hs) to 200 when establishing a bumped tunnel.

Bumped CONNECT requests are logged separately and we always use 200 status
code when bumping them. Eventually, tunnel and bumping code should use
real HttpReply objects instead of writing hard-coded reply strings that
logging code has no access to.

diff --git a/src/client_side_request.cc b/src/client_side_request.cc
index ef6640f73c3cf17f0ae332f4198532cb03e6e269..1ccc80711fbb90a5c5a41778f09e284dd0104fdb 100644 (file)
--- a/src/client_side_request.cc
+++ b/src/client_side_request.cc
@@ -1373,6 +1373,10 @@ ClientHttpRequest::sslBumpEstablish(comm_err_t errflag)
         return;
     }
 
+    // We lack HttpReply which logRequest() uses to log the status code.
+    // TODO: Use HttpReply instead of the "200 Connection established" string.
+    al.http.code = 200;
+
     getConn()->switchToHttps(request->GetHost(), request->port);
 }