# HG changeset patch
# User Goffi <goffi@goffi.org>
# Date 1636046856 -3600
# Node ID a806dcad6bfc32d40b6533a24cb7492b9cc7f45e
# Parent  842b522d525eb4f47a6e7d2fd48d8485627d55ca
core: don't callback `self.conn_deferred` if already called:

if the connection has been lost (e.g. forced disconnection by server),
`self.conn_deferred` may have already been called, thus this is now checked to avoid an
exception.

diff -r 842b522d525e -r a806dcad6bfc sat/core/xmpp.py
--- a/sat/core/xmpp.py	Thu Nov 04 18:13:55 2021 +0100
+++ b/sat/core/xmpp.py	Thu Nov 04 18:27:36 2021 +0100
@@ -302,7 +302,11 @@
         self.streamInitialized()
 
     def _finish_connection(self, __):
-        self.conn_deferred.callback(None)
+        if self.conn_deferred.called:
+            # can happen in case of forced disconnection by server
+            log.debug(f"{self} has already been connected")
+        else:
+            self.conn_deferred.callback(None)
 
     def streamInitialized(self):
         """Called after _authd"""