fixed clean_check of server_ids, fixed under-indented radarr get_movie,

and added __repr__ for cleaner logging

varken/dbmanager.py

@@ -2,7 +2,7 @@ import logging
 
 from influxdb import InfluxDBClient
 
-logger = logging.getLogger('Varken')
+logger = logging.getLogger('varken')
 
 class DBManager(object):
     def __init__(self, server):
@@ -16,5 +16,6 @@ class DBManager(object):
             self.influx.create_retention_policy('varken 30d/1h', '30d', '1', 'varken', False, '1h')
 
     def write_points(self, data):
-        logger.debug('Writing Data to InfluxDB {}'.format(data))
-        self.influx.write_points(data)
+        d = data
+        logger.debug('Writing Data to InfluxDB {}'.format(d))
+        self.influx.write_points(d)

varken/helpers.py

@@ -4,13 +4,14 @@ import tarfile
 import hashlib
 import geoip2.database
 import logging
-
+from functools import update_wrapper
 from json.decoder import JSONDecodeError
 from os.path import abspath, join
 from requests.exceptions import InvalidSchema, SSLError
 from urllib.request import urlretrieve
 
-logger = logging.getLogger('Varken')
+logger = logging.getLogger('varken')
+
 
 def geoip_download():
     tar_dbfile = abspath(join('.', 'data', 'GeoLite2-City.tar.gz'))

varken/iniparser.py

@@ -32,14 +32,33 @@ class INIParser(object):
 
         self.parse_opts()
 
-    def enable_check(self, type=None):
-        global_server_ids = self.config.get('global', type)
+    def enable_check(self, server_type=None):
+        t = server_type
+        global_server_ids = self.config.get('global', t)
         if global_server_ids.lower() in ['false', 'no', '0']:
-            logger.info('{} disabled.'.format(type.upper()))
+            logger.info('{} disabled.'.format(t.upper()))
             return False
         else:
-            logger.info('{} : ({})'.format(type.upper(), global_server_ids))
-            return global_server_ids
+            sids = self.clean_check(global_server_ids, t)
+            return sids
+
+    def clean_check(self, server_id_list, server_type=None):
+        t = server_type
+        sid_list = server_id_list
+        cleaned_list = sid_list.replace(' ', '').split(',')
+        valid_sids = []
+        for sid in cleaned_list:
+            try:
+                valid_sids.append(int(sid))
+            except ValueError:
+                logger.error("{} is not a valid server id number".format(sid))
+
+        if valid_sids:
+            logger.info('{} : {}'.format(t.upper(), valid_sids))
+            return valid_sids
+        else:
+            logger.error("No valid {}".format(t.upper()))
+            return False
 
     def read_file(self):
         file_path = join(self.data_folder, 'varken.ini')

varken/ombi.py

@@ -14,6 +14,9 @@ class OmbiAPI(object):
         self.session = Session()
         self.session.headers = {'Apikey': self.server.api_key}
 
+    def __repr__(self):
+        return "<ombi-{}>".format(self.server.id)
+
     def get_total_requests(self):
         self.now = datetime.now(timezone.utc).astimezone().isoformat()
         tv_endpoint = '/api/v1/Request/tv'

varken/radarr.py

@@ -14,6 +14,9 @@ class RadarrAPI(object):
         self.session = Session()
         self.session.headers = {'X-Api-Key': self.server.api_key}
 
+    def __repr__(self):
+        return "<radarr-{}>".format(self.server.id)
+
     def get_missing(self):
         endpoint = '/api/movie'
         self.now = datetime.now(timezone.utc).astimezone().isoformat()
@@ -29,8 +32,8 @@ class RadarrAPI(object):
         movies = [Movie(**movie) for movie in get]
 
         for movie in movies:
-            if self.server.get_missing:
-                if not movie.downloaded and movie.isAvailable:
+            if not movie.downloaded:
+                if movie.isAvailable:
                     ma = True
                 else:
                     ma = False

varken/sonarr.py

@@ -16,6 +16,9 @@ class SonarrAPI(object):
         self.session.headers = {'X-Api-Key': self.server.api_key}
         self.session.params = {'pageSize': 1000}
 
+    def __repr__(self):
+        return "<sonarr-{}>".format(self.server.id)
+
     def get_missing(self):
         endpoint = '/api/calendar'
         last_days = str(date.today() + timedelta(days=-self.server.missing_days))

varken/tautulli.py

@@ -19,6 +19,9 @@ class TautulliAPI(object):
         self.session.params['apikey'] = self.server.api_key
         self.endpoint = '/api/v2'
 
+    def __repr__(self):
+        return "<tautulli-{}>".format(self.server.id)
+
     def get_activity(self):
         self.now = datetime.now(timezone.utc).astimezone().isoformat()
         params = {'cmd': 'get_activity'}