From: Alex Rousskov <rousskov@measurement-factory.com>
Date: Wed, 1 Feb 2012 19:12:41 +0000 (-0700)
Subject: Set logged status code (%<Hs) to 200 when establishing a bumped tunnel.
X-Git-Tag: BumpSslServerFirst.take05~33
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=40d8be977bca50cbee6546e0b8b6ef1398634be9;p=thirdparty%2Fsquid.git

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 ef6640f73c..1ccc80711f 100644
--- 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);
 }