Only sync if subscribe succeeds

mautrix_signal/puppet.py

@@ -159,7 +159,7 @@ class Puppet(DBPuppet, BasePuppet):
             "first_name": names[0] if len(names) > 0 else "",
             "last_name": names[-1] if len(names) > 1 else "",
             "full_name": " ".join(names),
-            "phone": cls.fmt_phone(address.number) if address.number != None else None,
+            "phone": cls.fmt_phone(address.number) if address.number else None,
             "uuid": str(address.uuid) if address.uuid else None,
         }
         for pref in cls.config["bridge.displayname_preference"]:

mautrix_signal/signal.py

@@ -129,9 +129,9 @@ class SignalHandler(SignaldClient):
     async def start(self) -> None:
         await self.connect()
         async for user in u.User.all_logged_in():
-            # TODO handle errors
-            await self.subscribe(user.username)
-            self.loop.create_task(user.sync())
+            # TODO report errors to user?
+            if await self.subscribe(user.username):
+                self.loop.create_task(user.sync())
 
     async def stop(self) -> None:
         await self.disconnect()