feat: add fallback idp/mdl attribute mapping

Used when the primary mapping does not match against any particular user. This can be used in the case where attributes used for id management are transitioned from one field to another, and allows for a gradual non-disruptive rollover.
catalyst · Nov 4, 2024 · 2024110 · 2024110
1 parent 2dc645c
commit 2024110
Show file tree

Hide file tree

Showing 3 changed files with 66 additions and 22 deletions.
diff --git a/classes/auth.php b/classes/auth.php
@@ -671,8 +671,9 @@ public function saml_login_complete($attributes) {
         }
 
         $attr = $this->config->idpattr;
-        if (empty($attributes[$attr])) {
-            // Missing mapping IdP attribute. Login failed.
+        $attrsecondary = $this->config->idpattrsecondary;
+        if (empty($attributes[$attr]) && empty($attributes[$attrsecondary])) {
+            // Missing mapping IdP attribute (both primary and secondary). Login failed.
             $event = \core\event\user_login_failed::create(['other' => ['username' => 'unknown',
                 'reason' => AUTH_LOGIN_NOUSER]]);
             $event->trigger();
@@ -689,26 +690,15 @@ public function saml_login_complete($attributes) {
 
         // Find Moodle user.
         $user = false;
-        foreach ($attributes[$attr] as $uid) {
-            $insensitive = false;
-            $accentsensitive = true;
-            if ($this->config->tolower == saml2_settings::OPTION_TOLOWER_LOWER_CASE) {
-                $this->log(__FUNCTION__ . " to lowercase for $uid");
-                $uid = strtolower($uid);
-            }
-            if ($this->config->tolower == saml2_settings::OPTION_TOLOWER_CASE_INSENSITIVE) {
-                $this->log(__FUNCTION__ . " case insensitive compare for $uid");
-                $insensitive = true;
-            }
-            if ($this->config->tolower == saml2_settings::OPTION_TOLOWER_CASE_AND_ACCENT_INSENSITIVE) {
-                $this->log(__FUNCTION__ . " case and accent insensitive compare for $uid");
-                $insensitive = true;
-                $accentsensitive = false;
-            }
-            if ($user = user_extractor::get_user($this->config->mdlattr, $uid, $insensitive, $accentsensitive)) {
-                // We found a user.
-                break;
-            }
+
+        // Primary IdP attribute mapping.
+        if (!empty($attributes[$attr])) {
+            $user = $this->find_user_by_attributes($attributes[$attr], $this->config->mdlattr);
+        }
+
+        // Secondary IdP attribute mapping (if user not found yet).
+        if ($user === false && !empty($attributes[$attrsecondary])) {
+            $user = $this->find_user_by_attributes($attributes[$attrsecondary], $this->config->mdlattrsecondary);
         }
 
         // Moodle Workplace - Check IdP's tenant availability, for new user pre-allocate to tenant.
@@ -1343,4 +1333,40 @@ private function execute_callback($function, $file = 'lib.php') {
             }
         }
     }
+
+    /**
+     * Find and return a user matched using a list of provided attributes, against a Moodle field.
+     *
+     * Applies any case matching settings configured.
+     *
+     * @param array $idpattrs
+     * @param string $mdlattr
+     * @return mixed false if no user found, otherwise the user object.
+     */
+    private function find_user_by_attributes(array $idpattrs, string $mdlattr) {
+        $user = false;
+        foreach ($idpattrs as $uid) {
+            $insensitive = false;
+            $accentsensitive = true;
+            if ($this->config->tolower == saml2_settings::OPTION_TOLOWER_LOWER_CASE) {
+                $this->log(__FUNCTION__ . " to lowercase for $uid");
+                $uid = strtolower($uid);
+            }
+            if ($this->config->tolower == saml2_settings::OPTION_TOLOWER_CASE_INSENSITIVE) {
+                $this->log(__FUNCTION__ . " case insensitive compare for $uid");
+                $insensitive = true;
+            }
+            if ($this->config->tolower == saml2_settings::OPTION_TOLOWER_CASE_AND_ACCENT_INSENSITIVE) {
+                $this->log(__FUNCTION__ . " case and accent insensitive compare for $uid");
+                $insensitive = true;
+                $accentsensitive = false;
+            }
+            if ($user = user_extractor::get_user($mdlattr, $uid, $insensitive, $accentsensitive)) {
+                // We found a user.
+                break;
+            }
+        }
+        return $user;
+    }
+
 }
diff --git a/lang/en/auth_saml2.php b/lang/en/auth_saml2.php
@@ -95,6 +95,8 @@
 $string['flagresponsetype_help'] = 'If access is blocked based on configured group restrictions, how should Moodle respond?';
 $string['idpattr_help'] = 'Which IdP attribute should be matched against a Moodle user field?';
 $string['idpattr'] = 'Mapping IdP';
+$string['idpattrsecondary_help'] = 'When the primary IdP attribute does not match a user, map this field to the secondary Moodle mapped field.';
+$string['idpattrsecondary'] = 'Secondary Mapping IdP';
 $string['idpmetadata_badurl'] = 'Invalid metadata at {$a}';
 $string['idpmetadata_help'] = 'To use multiple IdPs enter each public metadata url on a new line.<br/>To override a name, place text before the http. eg. "Forced IdP Name http://ssp.local/simplesaml/saml2/idp/metadata.php"';
 $string['idpmetadata'] = 'IdP metadata xml OR public xml URL';
@@ -115,6 +117,8 @@
 $string['manageidpsheading'] = 'Manage available Identity Providers (IdPs)';
 $string['mdlattr_help'] = 'Which Moodle user field should the IdP attribute be matched to?';
 $string['mdlattr'] = 'Mapping Moodle';
+$string['mdlattrsecondary_help'] = 'Which Moodle user field should the IdP secondary attribute be matched to?';
+$string['mdlattrsecondary'] = 'Secondary Mapping Moodle';
 $string['wantassertionssigned'] = 'Want assertions signed';
 $string['wantassertionssigned_help'] = 'Whether assertions received by this SP must be signed';
 $string['assertionsconsumerservices'] = 'Assertions consumer services';

diff --git a/settings.php b/settings.php
@@ -314,6 +314,20 @@
             saml2_settings::OPTION_TOLOWER_EXACT,
             $toloweroptions));
 
+    // IDP attribute (secondary).
+    $settings->add(new admin_setting_configtext(
+            'auth_saml2/idpattrsecondary',
+            get_string('idpattrsecondary', 'auth_saml2'),
+            get_string('idpattrsecondary_help', 'auth_saml2'),
+            '', PARAM_TEXT));
+
+    // Moodle Field (secondary).
+    $settings->add(new admin_setting_configselect(
+            'auth_saml2/mdlattrsecondary',
+            get_string('mdlattrsecondary', 'auth_saml2'),
+            get_string('mdlattrsecondary_help', 'auth_saml2'),
+            '', user_fields::get_supported_fields()));
+
     // Requested Attributes.
     $settings->add(new admin_setting_configtextarea(
         'auth_saml2/requestedattributes',