Update listgroupsandmembers.py (#56)

Added test to skip groups if their id is a null strings

listgroupsandmembers.py

@@ -81,47 +81,49 @@ for group in groups:
     grouptype=group['type']
     groupproviderid=group['providerId']
 
-    # List the members of this group
+    if groupid!="": # Skip groups with empty id (this has been seen at least once at a customer site), because we cannot fetch their members.
-    endpoint='/identities/groups/'+groupid+'/members?limit=10000'
+    
-    method='get'
+        # List the members of this group
-    members_result_json=callrestapi(endpoint,method)
+        endpoint='/identities/groups/'+groupid+'/members?limit=10000'
-    if debug:
+        method='get'
-        print(members_result_json)
+        members_result_json=callrestapi(endpoint,method)
-        print('members_result_json is a '+type(members_result_json).__name__+' object') #members_result_json is a dict object
+        if debug:
-
+            print(members_result_json)
-    members=members_result_json['items']
+            print('members_result_json is a '+type(members_result_json).__name__+' object') #members_result_json is a dict object
-
+
-    for member in members:
+        members=members_result_json['items']
-        memberid=member['id']
+
-        membername=member['name']
+        for member in members:
-        membertype=member['type']
+            memberid=member['id']
-        memberproviderid=member['providerId']
+            membername=member['name']
-        user_email_string=''
+            membertype=member['type']
-        output=groupid+','+groupname+','+grouptype+','+groupproviderid+','+memberid+',"'+membername+'",'+membertype+','+memberproviderid
+            memberproviderid=member['providerId']
-
+            user_email_string=''
-        if show_email:
+            output=groupid+','+groupname+','+grouptype+','+groupproviderid+','+memberid+',"'+membername+'",'+membertype+','+memberproviderid
-            output=output+','
+
-
+            if show_email:
-        if membertype=='user' and show_email:
+                output=output+','
-
+
-            # List the members of this group
+            if membertype=='user' and show_email:
-            endpoint='/identities/users/'+memberid+'?limit=10000'
+
-            method='get'
+                # List the members of this group
-            user_details_json=callrestapi(endpoint,method)
+                endpoint='/identities/users/'+memberid+'?limit=10000'
-            if debug:
+                method='get'
-                print(user_details_json)
+                user_details_json=callrestapi(endpoint,method)
-                print('user_details_json is a '+type(user_details_json).__name__+' object') #user_details_json is a dict object
+                if debug:
-
+                    print(user_details_json)
-            if 'emailAddresses' in user_details_json:
+                    print('user_details_json is a '+type(user_details_json).__name__+' object') #user_details_json is a dict object
-                user_emails=user_details_json['emailAddresses']
+
-                user_email_string=''
+                if 'emailAddresses' in user_details_json:
-
+                    user_emails=user_details_json['emailAddresses']
-                for email in user_emails:
+                    user_email_string=''
-                    email_address=email['value']
+
-                    if user_email_string!='':
+                    for email in user_emails:
-                        user_email_string=user_email_string+';'
+                        email_address=email['value']
-                    user_email_string=user_email_string+email_address
+                        if user_email_string!='':
-
+                            user_email_string=user_email_string+';'
-                output=output+user_email_string
+                        user_email_string=user_email_string+email_address
-
+
-        print(output)
+                    output=output+user_email_string
+
+            print(output)