📧 fix: LDAP login after User verification changes (#3003)

danny-avila · Jun 7, 2024 · baf0848 · baf0848
1 parent 1da9211
commit baf0848
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/api/strategies/ldapStrategy.js b/api/strategies/ldapStrategy.js
@@ -1,5 +1,5 @@
 const LdapStrategy = require('passport-ldapauth');
-const { findUser, createUser } = require('~/models/userMethods');
+const { findUser, createUser, updateUser } = require('~/models/userMethods');
 const fs = require('fs');
 
 const ldapOptions = {
@@ -49,14 +49,16 @@ const ldapLogin = new LdapStrategy(ldapOptions, async (userinfo, done) => {
         emailVerified: true,
         name: fullName,
       };
+      const userId = await createUser(user);
+      user._id = userId;
     } else {
       user.provider = 'ldap';
       user.ldapId = userinfo.uid;
       user.username = username;
       user.name = fullName;
     }
 
-    await createUser(user);
+    user = await updateUser(user._id, user);
 
     done(null, user);
   } catch (err) {