From: Francesco Chemolli <5175948+kinkie@users.noreply.github.com>
Date: Sat, 18 Nov 2023 06:09:44 +0000 (+0000)
Subject: Refactor Security::BlindPeerConnector constructor (#1511)
X-Git-Tag: SQUID_7_0_1~283
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=2c6781d8b54a93b89e4395e65feba77f985beb94;p=thirdparty%2Fsquid.git

Refactor Security::BlindPeerConnector constructor (#1511)

Avoid having to include HttpRequest.h in
BlindPeerConnector.h by moving the
definition of the constructor to the .cc file
---

diff --git a/src/security/BlindPeerConnector.cc b/src/security/BlindPeerConnector.cc
index 7372df9608..1811ae96f6 100644
--- a/src/security/BlindPeerConnector.cc
+++ b/src/security/BlindPeerConnector.cc
@@ -86,3 +86,13 @@ Security::BlindPeerConnector::noteNegotiationDone(ErrorState *error)
     }
 }
 
+Security::BlindPeerConnector::BlindPeerConnector(HttpRequestPointer &aRequest,
+        const Comm::ConnectionPointer &aServerConn,
+        const AsyncCallback<EncryptorAnswer> &aCallback,
+        const AccessLogEntryPointer &alp,
+        time_t timeout) :
+    AsyncJob("Security::BlindPeerConnector"),
+    Security::PeerConnector(aServerConn, aCallback, alp, timeout)
+{
+    request = aRequest;
+}
diff --git a/src/security/BlindPeerConnector.h b/src/security/BlindPeerConnector.h
index 62e19c77f5..0001579c7b 100644
--- a/src/security/BlindPeerConnector.h
+++ b/src/security/BlindPeerConnector.h
@@ -10,7 +10,6 @@
 #define SQUID_SRC_SECURITY_BLINDPEERCONNECTOR_H
 
 #include "http/forward.h"
-#include "HttpRequest.h"
 #include "security/PeerConnector.h"
 
 class ErrorState;
@@ -26,12 +25,7 @@ public:
                        const Comm::ConnectionPointer &aServerConn,
                        const AsyncCallback<EncryptorAnswer> &aCallback,
                        const AccessLogEntryPointer &alp,
-                       const time_t timeout = 0) :
-        AsyncJob("Security::BlindPeerConnector"),
-        Security::PeerConnector(aServerConn, aCallback, alp, timeout)
-    {
-        request = aRequest;
-    }
+                       time_t timeout = 0);
 
     /* Security::PeerConnector API */
 
diff --git a/src/tests/stub_libsecurity.cc b/src/tests/stub_libsecurity.cc
index 553253612d..e6871e0296 100644
--- a/src/tests/stub_libsecurity.cc
+++ b/src/tests/stub_libsecurity.cc
@@ -18,6 +18,14 @@
 CBDATA_NAMESPACED_CLASS_INIT(Security, BlindPeerConnector);
 namespace Security
 {
+BlindPeerConnector::BlindPeerConnector(HttpRequestPointer &, const Comm::ConnectionPointer & aServerConn,
+                                       const AsyncCallback<EncryptorAnswer> & aCallback,
+                                       const AccessLogEntryPointer &alp,
+                                       time_t) :
+    AsyncJob("Security::BlindPeerConnector"),
+    Security::PeerConnector(aServerConn, aCallback, alp, 0)
+{STUB_NOP}
+
 bool BlindPeerConnector::initialize(Security::SessionPointer &) STUB_RETVAL(false)
 Security::ContextPointer BlindPeerConnector::getTlsContext() STUB_RETVAL(Security::ContextPointer())
 void BlindPeerConnector::noteNegotiationDone(ErrorState *) STUB