From 5e1f3c2ecfd141fb97b91ec1c0e068b916be0c46 Mon Sep 17 00:00:00 2001 From: Michael Johnson Date: Tue, 26 Jun 2018 09:06:32 -0700 Subject: [PATCH] Display AD Login Info Not Static String MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Rather than displaying the string “Azure AD” for all users with connections to AAD, set the “Provider Login” value to the Displayable ID from the AD info. Fixes the new requirement for a unique value in SonarQube 7.2 and makes it easier to see what AD account is associated with a specific SQ user. --- src/main/java/org/almrangers/auth/aad/AadIdentityProvider.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/java/org/almrangers/auth/aad/AadIdentityProvider.java b/src/main/java/org/almrangers/auth/aad/AadIdentityProvider.java index b1c1efd..4018da8 100644 --- a/src/main/java/org/almrangers/auth/aad/AadIdentityProvider.java +++ b/src/main/java/org/almrangers/auth/aad/AadIdentityProvider.java @@ -127,7 +127,7 @@ public void callback(CallbackContext context) { UserInfo aadUser = result.getUserInfo(); UserIdentity.Builder userIdentityBuilder = UserIdentity.builder() - .setProviderLogin(getName()) + .setProviderLogin(aadUser.getDisplayableId()) .setLogin(getLogin(aadUser)) .setName(aadUser.getGivenName() + " " + aadUser.getFamilyName()) .setEmail(aadUser.getDisplayableId());