Skip to content

Commit

Permalink
Add more group sync logging
Browse files Browse the repository at this point in the history
  • Loading branch information
Weves committed Jan 30, 2025
1 parent a70d472 commit 39b3a50
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions backend/ee/onyx/external_permissions/confluence/group_sync.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@ def _build_group_member_email_map(
) -> dict[str, set[str]]:
group_member_emails: dict[str, set[str]] = {}
for user_result in confluence_client.paginated_cql_user_retrieval():
logger.debug(f"Processing groups for user: {user_result}")

user = user_result.get("user", {})
if not user:
logger.warning(f"user result missing user field: {user_result}")
Expand All @@ -33,10 +35,17 @@ def _build_group_member_email_map(
logger.warning(f"user result missing email field: {user_result}")
continue

all_users_groups: set[str] = set()
for group in confluence_client.paginated_groups_by_user_retrieval(user):
# group name uniqueness is enforced by Confluence, so we can use it as a group ID
group_id = group["name"]
group_member_emails.setdefault(group_id, set()).add(email)
all_users_groups.add(group_id)

if not group_member_emails:
logger.warning(f"No groups found for user with email: {email}")
else:
logger.debug(f"Found groups {all_users_groups} for user with email {email}")

return group_member_emails

Expand Down

0 comments on commit 39b3a50

Please sign in to comment.