diff --git a/src/externalldap.js b/src/externalldap.js index b71e214ff..68f8544a1 100644 --- a/src/externalldap.js +++ b/src/externalldap.js @@ -188,12 +188,12 @@ function createAndVerifyUserIfNotExist(identifier, password, callback) { if (externalLdapConfig.provider === 'noop') return callback(new BoxError(BoxError.BAD_STATE, 'not enabled')); if (!externalLdapConfig.autoCreate) return callback(new BoxError(BoxError.BAD_STATE, 'auto create not enabled')); - ldapSearch(externalLdapConfig, { filter: `${externalLdapConfig.usernameField}=${identifier}` }, function (error, users) { + ldapSearch(externalLdapConfig, { filter: `${externalLdapConfig.usernameField}=${identifier}` }, function (error, ldapUsers) { if (error) return callback(error); - if (users.length === 0) return callback(new BoxError(BoxError.NOT_FOUND)); - if (users.length > 1) return callback(new BoxError(BoxError.CONFLICT)); + if (ldapUsers.length === 0) return callback(new BoxError(BoxError.NOT_FOUND)); + if (ldapUsers.length > 1) return callback(new BoxError(BoxError.CONFLICT)); - let user = translateUser(externalLdapConfig, users[0]); + let user = translateUser(externalLdapConfig, ldapUsers[0]); if (!validUserRequirements(user)) return callback(new BoxError(BoxError.BAD_FIELD)); users.create(user.username, null /* password */, user.email, user.displayName, { source: 'ldap' }, auditSource.EXTERNAL_LDAP_AUTO_CREATE, function (error, user) {