fix groups in azure

zvonand · zvonand · commit d769e8ab7ad6 · 2025-09-22T11:46:28.000+02:00
diff --git a/src/Access/TokenProcessorsOpaque.cpp b/src/Access/TokenProcessorsOpaque.cpp
@@ -224,7 +224,10 @@ bool AzureTokenProcessor::resolveAndValidate(const TokenCredentials & credential
             }
 
             auto group_data = group.get<picojson::object>();
-            String group_name = getValueByKey(group_data, "id");
+            if (!group_data.contains("displayName"))
+                continue;
+
+            String group_name = getValueByKey(group_data, "displayName");
             external_groups_names.insert(group_name);
             LOG_TRACE(getLogger("TokenAuthentication"), "{}: User {}: new external group {}", processor_name, credentials.getUserName(), group_name);
         }

Original file line number	Diff line number	Diff line change
`@@ -224,7 +224,10 @@ bool AzureTokenProcessor::resolveAndValidate(const TokenCredentials & credential`
`224`	`224`	`}`
`225`	`225`
`226`	`226`	`auto group_data = group.get<picojson::object>();`
`227`		`- String group_name = getValueByKey(group_data, "id");`
	`227`	`+ if (!group_data.contains("displayName"))`
	`228`	`+ continue;`
	`229`	`+`
	`230`	`+ String group_name = getValueByKey(group_data, "displayName");`
`228`	`231`	`external_groups_names.insert(group_name);`
`229`	`232`	`LOG_TRACE(getLogger("TokenAuthentication"), "{}: User {}: new external group {}", processor_name, credentials.getUserName(), group_name);`
`230`	`233`	`}`