MDL-27107 auth_mnet - consider all incoming roaming users as confirmed

The 'confirmed' field can't be optional. If the identity provider did
not export it, the new remote user would be created with 'confirmed' set
to default zero and as such could be a subject of scheduled deletion.
This commit is contained in:
David Mudrak
2011-08-28 02:28:12 +02:00
parent 58609afebb
commit a1e52c70e5
2 changed files with 2 additions and 1 deletions
+1
View File
@@ -276,6 +276,7 @@ class auth_plugin_mnet extends auth_plugin_base {
*/
$remoteuser->mnethostid = $remotehost->id;
$remoteuser->firstaccess = time(); // First time user in this server, grab it here
$remoteuser->confirmed = 1;
$remoteuser->id = $DB->insert_record('user', $remoteuser);
$firsttime = true;
+1 -1
View File
@@ -586,6 +586,7 @@ function mnet_profile_field_options() {
'timemodified', // will be set to relative to the host anyway
'auth', // going to be set to 'mnet'
'deleted', // we should never get deleted users sent over, but don't send this anyway
'confirmed', // unconfirmed users can't log in to their home site, all remote users considered confirmed
'password', // no password for mnet users
'theme', // handled separately
'lastip', // will be set to relative to the host anyway
@@ -610,7 +611,6 @@ function mnet_profile_field_options() {
'username',
'email',
'auth',
'confirmed',
'deleted',
'firstname',
'lastname',