From 366c78a33f63ac4412cc35d1a77386cae4a163a3 Mon Sep 17 00:00:00 2001
From: Eduard Bagdasaryan <eduard.bagdasaryan@measurement-factory.com>
Date: Fri, 13 Aug 2021 19:43:46 +0300
Subject: [PATCH] ICAP: do not set connect_timeout on the established
 connection

Icap::Xaction code should not bother with connect_timeout at all:
this work is performed by connection establishing jobs
such as ConnOpener and Ssl::IcapPeerConnector.

Moreover, the Xaction::noteCommTimedout() handler does not expect
disconnected Connection and should not be used while connecting.
---
 src/adaptation/icap/Xaction.cc | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/src/adaptation/icap/Xaction.cc b/src/adaptation/icap/Xaction.cc
index ce7a4dce06..a8b568afe2 100644
--- a/src/adaptation/icap/Xaction.cc
+++ b/src/adaptation/icap/Xaction.cc
@@ -294,11 +294,6 @@ Adaptation::Icap::Xaction::useIcapConnection(const Comm::ConnectionPointer &conn
     connection = conn;
     service().noteConnectionUse(connection);
 
-    typedef CommCbMemFunT<Adaptation::Icap::Xaction, CommTimeoutCbParams> TimeoutDialer;
-    AsyncCall::Pointer timeoutCall =  asyncCall(93, 5, "Adaptation::Icap::Xaction::noteCommTimedout",
-                                      TimeoutDialer(this,&Adaptation::Icap::Xaction::noteCommTimedout));
-    commSetConnTimeout(connection, TheConfig.connect_timeout(service().cfg().bypass), timeoutCall);
-
     typedef CommCbMemFunT<Adaptation::Icap::Xaction, CommCloseCbParams> CloseDialer;
     closer =  asyncCall(93, 5, "Adaptation::Icap::Xaction::noteCommClosed",
                         CloseDialer(this,&Adaptation::Icap::Xaction::noteCommClosed));
-- 
2.47.3