comments

Author: tomklapiscak

src/mas/devops/users.py

@@ -19,16 +19,6 @@
 import time
 import re
 
-'''
-TODO:
-    MVI / other apps?
-    unit tests
-
-    how to cope with users that have been soft-deleted? tolerate / skip - ensure they are removed from secret so
-    we don't get caught in an infinite loop?
-
-'''
-
 
 class MASUserUtils():
     '''
@@ -324,7 +314,7 @@ def link_user_to_local_idp(self, user_id, email_password=False):
             "emailPassword": email_password
         }
         payload = {
-            "idpUserId": user_id
+            "idpUserId": user_id,
         }
         headers = {
             "Content-Type": "application/json",
@@ -506,10 +496,6 @@ def create_or_get_manage_api_key_for_user(self, user_id, temporary=False):
             "lean": 1,
         }
 
-        # TODO: is this just a temporary API Key for this script?
-        #       if so, add cleanup logic (atext) and perhaps set an expiration date in case cleanup fails
-
-        #
         payload = {
             "expiration": -1,
             "userid": user_id