Philipp Hörist pushed to branch master at gajim / gajim


Commits:
379fc572 by Philipp Hörist at 2022-10-16T19:55:22+02:00
fix: Client: Load trusted certificates on connect()

This eliminates some race conditions where _create_client() is called
before the cert is stored.

Fixes #11220

- - - - -


1 changed file:

- gajim/common/client.py


Changes:

=====================================
gajim/common/client.py
=====================================
@@ -174,8 +174,6 @@ def _create_client(self) -> None:
             self.password = passwords.get_password(self._account)
 
         self._client.set_password(self.password)
-        self._client.set_accepted_certificates(
-            app.cert_store.get_certificates())
 
         self._client.subscribe('resume-failed', self._on_resume_failed)
         self._client.subscribe('resume-successful', self._on_resume_successful)
@@ -544,6 +542,8 @@ def connect(self, ignored_tls_errors: IgnoredTlsErrorsT = 
None) -> None:
 
         log.info('Connect')
 
+        self._client.set_accepted_certificates(
+            app.cert_store.get_certificates())
         self._client.set_ignored_tls_errors(ignored_tls_errors)
         self._reconnect = True
         self._disable_reconnect_timer()



View it on GitLab: 
https://dev.gajim.org/gajim/gajim/-/commit/379fc572fc67d1eda3b15a6a67f12dd45a9d7ee2

-- 
View it on GitLab: 
https://dev.gajim.org/gajim/gajim/-/commit/379fc572fc67d1eda3b15a6a67f12dd45a9d7ee2
You're receiving this email because of your account on dev.gajim.org.


_______________________________________________
Commits mailing list
[email protected]
https://lists.gajim.org/cgi-bin/listinfo/commits

Reply via email to