Plugin: Azure AD: Diversify exception to be more explicit regarding e…

…mpty fields in Azure profile - refs BT#21739
ywarnier · May 28, 2024 · 0c299d9 · 0c299d9
1 parent 4a3624a
commit 0c299d9
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/plugin/azure_active_directory/src/callback.php b/plugin/azure_active_directory/src/callback.php
@@ -64,8 +64,11 @@
 
     // We use the e-mail to authenticate the user, so check that at least one
     // e-mail source exists
-    if (empty($me['mail']) || empty($me['mailNickname'])) {
-        throw new Exception('Mail empty');
+    if (empty($me['mail'])) {
+        throw new Exception('The mail field is empty in Azure AD and is needed to set the organisation email for this user.');
+    }
+    if (empty($me['mailNickname'])) {
+        throw new Exception('The mailNickname field is empty in Azure AD and is needed to set the unique Azure ID for this user.');
     }
 
     $extraFieldValue = new ExtraFieldValue('user');