Merge pull request #65 from rotdrop/bugfix/reconnect-really

IMHO, "reconnect" should catch __ALL__ sorts of connection errors.
azlux · May 10, 2020 · 8f017f7 · 8f017f7
2 parents bd31886 + 88aec39
commit 8f017f7
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/pymumble_py3/mumble.py b/pymumble_py3/mumble.py
@@ -113,8 +113,12 @@ def run(self):
 
             if self.connect() >= PYMUMBLE_CONN_STATE_FAILED:  # some error occurred, exit here
                 self.ready_lock.release()
-                raise ConnectionRejectedError ("Connection error with the Mumble (murmur) Server")
-                break
+                if not self.reconnect or not self.parent_thread.is_alive():
+                    raise ConnectionRejectedError ("Connection error with the Mumble (murmur) Server")
+                    break
+                else:
+                    time.sleep(PYMUMBLE_CONNECTION_RETRY_INTERVAL)
+                    continue
 
             try:
                 self.loop()