Skip to content

Commit

Permalink
Display AD Login Info Not Static String
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
srvrguy committed Jun 26, 2018
1 parent 8d97fc5 commit 5e1f3c2
Showing 1 changed file with 1 addition and 1 deletion.
Expand Up @@ -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());
Expand Down

0 comments on commit 5e1f3c2

Please sign in to comment.