Skip to content

Commit

Permalink
Merge pull request #147 from dustinrue/master
Browse files Browse the repository at this point in the history
The UPN property doesn't exist, but OID does
  • Loading branch information
psignoret authored Aug 24, 2017
2 parents f3f5781 + 07c1e7e commit cc48018
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion aad-sso-wordpress.php
Original file line number Diff line number Diff line change
Expand Up @@ -296,7 +296,7 @@ function authenticate( $user, $username, $password ) {
// exists in WordPress (either because it already existed, or we created it
// on-the-fly). All that's left is to set the roles based on group membership.
if ( true === $this->settings->enable_aad_group_to_wp_role ) {
$user = $this->update_wp_user_roles( $user, $jwt->upn, $jwt->tid );
$user = $this->update_wp_user_roles( $user, $jwt->oid, $jwt->tid );
}
}

Expand Down

0 comments on commit cc48018

Please sign in to comment.