small fix

uc-cdis · Feb 16, 2021 · f38e659 · f38e659
1 parent 5feaee9
commit f38e659
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/fence/job/visa_update_cronjob.py b/fence/job/visa_update_cronjob.py
@@ -171,7 +171,7 @@ def _pick_client(self, visa):
             )
         if not client:
             raise Exception(
-                "Visa Client not set up or not avaialable for type {}".format(visa.type)
+                "Visa Client not set up or not available for type {}".format(visa.type)
             )
         return client
 

diff --git a/fence/sync/sync_users.py b/fence/sync/sync_users.py
@@ -1737,8 +1737,19 @@ def _pick_sync_type(self, visa):
         """
         Pick type of visa to parse according to the visa provider
         """
+        sync_client = None
         if visa.type in self.visa_types["ras"]:
-            return self.ras_sync_client
+            sync_client = self.ras_sync_client
+        else:
+            raise Exception(
+                "Visa type {} not recognized. Configure in fence-config".format(
+                    visa.type
+                )
+            )
+        if not sync_client:
+            raise Exception("Sync client for {} not configured".format(visa.type))
+
+        return sync_client
 
     def parse_user_visas(self, db_session):
         """