diff --git a/src/externalldap.js b/src/externalldap.js index ff22dd73e..b68da23b3 100644 --- a/src/externalldap.js +++ b/src/externalldap.js @@ -158,7 +158,7 @@ async function clientSearch(client, dn, searchOptions) { if (error instanceof ldap.NoSuchObjectError) return reject(new BoxError(BoxError.NOT_FOUND)); if (error) return reject(new BoxError(BoxError.EXTERNAL_ERROR, error)); - let ldapObjects = []; + const ldapObjects = []; result.on('searchEntry', entry => ldapObjects.push(entry.object)); result.on('error', error => reject(new BoxError(BoxError.EXTERNAL_ERROR, error))); @@ -348,11 +348,11 @@ async function syncUsers(config, progressCallback) { debug(`syncUsers: Found ${ldapUsers.length} users`); let percent = 10; - let step = 30/(ldapUsers.length+1); // ensure no divide by 0 + const step = 30/(ldapUsers.length+1); // ensure no divide by 0 // we ignore all errors here and just log them for now for (let i = 0; i < ldapUsers.length; i++) { - let ldapUser = translateUser(config, ldapUsers[i]); + const ldapUser = translateUser(config, ldapUsers[i]); if (!validUserRequirements(ldapUser)) continue; percent += step; @@ -399,7 +399,7 @@ async function syncGroups(config, progressCallback) { debug(`syncGroups: Found ${ldapGroups.length} groups`); let percent = 40; - let step = 30/(ldapGroups.length+1); // ensure no divide by 0 + const step = 30/(ldapGroups.length+1); // ensure no divide by 0 for (const ldapGroup of ldapGroups) { let groupName = ldapGroup[config.groupnameField]; @@ -453,7 +453,7 @@ async function syncGroupMembers(config, progressCallback) { } // since our group names are lowercase we cannot use potentially case matching ldap filters - let found = result.find(function (r) { + const found = result.find(function (r) { if (!r[config.groupnameField]) return false; return r[config.groupnameField].toLowerCase() === group.name; });