Skip to content

Commit cc48018

Browse files
authored
Merge pull request #147 from dustinrue/master
The UPN property doesn't exist, but OID does
2 parents f3f5781 + 07c1e7e commit cc48018

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

aad-sso-wordpress.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -296,7 +296,7 @@ function authenticate( $user, $username, $password ) {
296296
// exists in WordPress (either because it already existed, or we created it
297297
// on-the-fly). All that's left is to set the roles based on group membership.
298298
if ( true === $this->settings->enable_aad_group_to_wp_role ) {
299-
$user = $this->update_wp_user_roles( $user, $jwt->upn, $jwt->tid );
299+
$user = $this->update_wp_user_roles( $user, $jwt->oid, $jwt->tid );
300300
}
301301
}
302302

0 commit comments

Comments
 (0)