Use the main threadpool to disconnect API clients

refs #8554

diff --git a/lib/remote/apiclient.cpp b/lib/remote/apiclient.cpp
index cf33aec4a0f06bf875e6615d71f48e21012d9da2..69dee98bdda932ef6f43818db4b20b378d222455 100644 (file)
--- a/lib/remote/apiclient.cpp
+++ b/lib/remote/apiclient.cpp
@@ -258,11 +258,8 @@ Value RequestCertificateHandler(const MessageOrigin& origin, const Dictionary::P
 void ApiClient::TimeoutTimerHandler(void)
 {
        if (m_Seen < Utility::GetTime() - 60 && (!m_Endpoint || !m_Endpoint->GetSyncing())) {
-               /* Obtain a strong reference to ourselves because Disconnect otherwise removes the last reference */
-               ApiClient::Ptr self = this;
-
                Log(LogInformation, "ApiClient")
                    <<  "No messages for identity '" << m_Identity << "' have been received in the last 60 seconds.";
-               Disconnect();
+               Utility::QueueAsyncCallback(boost::bind(&ApiClient::Disconnect, ApiClient::Ptr(this)));
        }
 }