Skip to content
This repository has been archived by the owner on May 14, 2024. It is now read-only.

It throws exception when Ldap Server returns 9 error code (LDAP_PARTIAL_RESULTS) #366

Closed
gaupoit opened this issue Apr 4, 2016 · 1 comment
Labels

Comments

@gaupoit
Copy link

gaupoit commented Apr 4, 2016

Our LDAP server returns error code is 9 and I looked at the codebase. I don't know why it is not in codes.js and it will cause an error here

module.exports.getError = function (res) {
assert.ok(res instanceof LDAPResult, 'res (LDAPResult) required');

var errObj = ERRORS[res.status];
==> var E = module.exports[errObj.err];
return new E(res.errorMessage || errObj.message,
res.matchedDN || null,
module.exports.getError);
};

because errObj is undefined. Should we add LDAP_PARTIAL_RESULTS: 9 in codes.js?

@jsumners
Copy link
Member

👋

On February 22, 2023, we released version 3 of this library. As a result, we are closing this issue/pull request.

Please see issue #839 for more information, including how to proceed if you feel this closure is in error.

@ldapjs ldapjs locked as resolved and limited conversation to collaborators Feb 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants