Skip to content

Commit e61ec81

Browse files
committed
fix issue with all users setting
1 parent 15a228b commit e61ec81

File tree

1 file changed

+9
-5
lines changed

1 file changed

+9
-5
lines changed

user_sync/engine/umapi.py

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -385,6 +385,9 @@ def read_desired_user_groups(self, mappings, directory_connector):
385385
continue
386386

387387
self.filtered_directory_user_index.add(directory_user)
388+
umapi_info = self.get_umapi_info(PRIMARY_TARGET_NAME)
389+
umapi_info.add_desired_group_for(directory_user['identity_type'], directory_user['domain'],
390+
directory_user['email'], directory_user['username'], None)
388391

389392
# set up groups in hook scope; the target groups will be used whether or not there's customer hook code
390393
self.after_mapping_hook_scope['source_groups'] = set()
@@ -1326,14 +1329,15 @@ def add_desired_group_for(self, id_type, domain, email, username, group):
13261329
:type user_key: str
13271330
:type group: Optional(str)
13281331
"""
1329-
if group is None:
1330-
return
1331-
1332-
normalized_group_name = normalize_string(group)
1332+
if group is not None:
1333+
normalized_group_name = normalize_string(group)
1334+
else:
1335+
normalized_group_name = None
13331336
desired_groups_rec = self.get_desired_groups(email, username)
13341337
if desired_groups_rec is None:
13351338
groups = set()
1336-
groups.add(normalized_group_name)
1339+
if normalized_group_name is not None:
1340+
groups.add(normalized_group_name)
13371341
desired_groups_rec = {
13381342
'id_type': id_type,
13391343
'domain': domain,

0 commit comments

Comments
 (0)