fix(ingest) Azure AD: support nested groups (#4367)

datahub-project · Mar 12, 2022 · ad45082 · ad45082
1 parent b54d74b
commit ad45082
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/metadata-ingestion/src/datahub/ingestion/source/identity/azure_ad.py b/metadata-ingestion/src/datahub/ingestion/source/identity/azure_ad.py
@@ -202,8 +202,8 @@ def _add_group_members_to_group_membership(
                 odata_type = azure_ad_member.get("@odata.type")
                 if odata_type == "#microsoft.graph.user":
                     self._add_user_to_group_membership(
-                        azure_ad_member,
                         parent_corp_group_urn,
+                        azure_ad_member,
                         user_urn_to_group_membership,
                     )
                 elif odata_type == "#microsoft.graph.group":
@@ -220,8 +220,8 @@ def _add_group_members_to_group_membership(
 
     def _add_user_to_group_membership(
         self,
-        azure_ad_user: dict,
         group_urn: str,
+        azure_ad_user: dict,
         user_urn_to_group_membership: Dict[str, GroupMembershipClass],
     ) -> None:
         user_urn = self._map_azure_ad_user_to_urn(azure_ad_user)