trustchain-oc1-orchestral/authentik
Archived
10
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

sources/oauth: use UPN for username with azure AD source

Browse source 
closes #3468
breaking

Signed-off-by: Jens Langhammer <jens.langhammer@beryju.org>
This commit is contained in:
Jens Langhammer 2022-08-23 20:55:25 +02:00
parent e994a01e80
commit 13a302cdad
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
authentik/sources/oauth/tests/test_type_azure_ad.py
View file

@ -41,6 +41,6 @@ class TestTypeAzureAD(TestCase):
def test_enroll_context(self):
"""Test azure_ad Enrollment context"""
ak_context = AzureADOAuthCallback().get_user_enroll_context(AAD_USER)
self.assertEqual(ak_context["username"], AAD_USER["displayName"])
self.assertEqual(ak_context["username"], AAD_USER["userPrincipalName"])
self.assertEqual(ak_context["email"], AAD_USER["mail"])
self.assertEqual(ak_context["name"], AAD_USER["displayName"])

2
authentik/sources/oauth/types/azure_ad.py
View file

@ -31,7 +31,7 @@ class AzureADOAuthCallback(OAuthCallback):
) -> dict[str, Any]:
mail = info.get("mail", None) or info.get("otherMails", [None])[0]
return {
"username": info.get("displayName"),
"username": info.get("userPrincipalName"),
"email": mail,
"name": info.get("displayName"),
}