Skip to content

Commit

Permalink
Merge pull request #33 from shaozi/issue-30
Browse files Browse the repository at this point in the history
fixed issue #30
  • Loading branch information
shaozi authored Aug 15, 2022
2 parents b321dda + 9c3a337 commit 25de126
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 5 deletions.
11 changes: 7 additions & 4 deletions index.js
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,12 @@ async function _searchUser(
user = entry.object
})
res.on('searchReference', function (referral) {
console.log('referral: ' + referral.uris.join())
// TODO: we don't support reference yet
// If the server was able to locate the entry referred to by the baseObject
// but could not search one or more non-local entries,
// the server may return one or more SearchResultReference messages,
// each containing a reference to another set of servers for continuing the operation.
// referral.uris
})
res.on('error', function (err) {
reject(err)
Expand Down Expand Up @@ -133,9 +138,7 @@ async function _searchUserGroups(
res.on('searchEntry', function (entry) {
groups.push(entry.object)
})
res.on('searchReference', function (referral) {
console.log('referral: ' + referral.uris.join())
})
res.on('searchReference', function (referral) {})
res.on('error', function (err) {
reject(err)
ldapClient.unbind()
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "ldap-authentication",
"version": "2.3.0",
"version": "2.3.1",
"description": "A simple async nodejs library for LDAP user authentication",
"main": "index.js",
"types": "./index.d.ts",
Expand Down

0 comments on commit 25de126

Please sign in to comment.