Improved how Bazarr deals with Sonarr SignalR feed issues that are raising exceptions.

bazarr/main.py

@@ -204,10 +204,12 @@ def proxy(protocol, url):
             return dict(status=False, error=result.raise_for_status())
 
 
+greenlets = []
 if settings.general.getboolean('use_sonarr'):
-    Greenlet.spawn(sonarr_signalr_client.start)
+    greenlets.append(Greenlet.spawn(sonarr_signalr_client.start))
 if settings.general.getboolean('use_radarr'):
-    Greenlet.spawn(radarr_signalr_client.start)
+    greenlets.append(Greenlet.spawn(radarr_signalr_client.start))
+gevent.joinall(greenlets)
 
 
 if __name__ == "__main__":

bazarr/signalr_client.py

@@ -33,11 +33,12 @@ class SonarrSignalrClient:
         self.connection = None
 
     def start(self):
-        if get_sonarr_version().startswith('2.'):
+        sonarr_version = get_sonarr_version()
+        if sonarr_version.startswith('2.'):
             logging.warning('BAZARR can only sync from Sonarr v3 SignalR feed to get real-time update. You should '
-                            'consider upgrading.')
+                            'consider upgrading your version({}).'.format(sonarr_version))
-            return
+            raise gevent.GreenletExit
-
+        else:
             logging.info('BAZARR trying to connect to Sonarr SignalR feed...')
             self.configure()
             while not self.connection.started:
@@ -46,10 +47,13 @@ class SonarrSignalrClient:
                 except ConnectionError:
                     gevent.sleep(5)
                 except json.decoder.JSONDecodeError:
-                logging.error("BAZARR cannot parse JSON returned by SignalR feed. This is a known issue when Sonarr "
+                    logging.error("BAZARR cannot parse JSON returned by SignalR feed. This is a known issue when "
-                              "doesn't have write permission to it's /config/xdg directory.")
+                                  "Sonarr have issue accessing it's /config/xdg directory. You should delete that "
-                self.stop()
+                                  "directory and restart Sonarr.")
+                    raise gevent.GreenletExit
+                else:
                     logging.info('BAZARR SignalR client for Sonarr is connected and waiting for events.')
+                finally:
                     if not args.dev:
                         scheduler.add_job(update_series, kwargs={'send_event': True}, max_instances=1)
                         scheduler.add_job(sync_episodes, kwargs={'send_event': True}, max_instances=1)