Skip to content

Commit

Permalink
refactor: Upgrade ldapjs from 2.3.3 to 3.0.7 (#8947)
Browse files Browse the repository at this point in the history
  • Loading branch information
mtrezza authored Feb 26, 2024
1 parent 68b745a commit f5d6fc9
Show file tree
Hide file tree
Showing 3 changed files with 200 additions and 44 deletions.
240 changes: 198 additions & 42 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@
"intersect": "1.0.1",
"jsonwebtoken": "9.0.0",
"jwks-rsa": "3.1.0",
"ldapjs": "2.3.3",
"ldapjs": "3.0.7",
"lodash": "4.17.21",
"lru-cache": "10.1.0",
"mime": "3.0.0",
Expand Down
2 changes: 1 addition & 1 deletion src/Adapters/Auth/ldap.js
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,7 @@ function searchForGroup(client, options, id, resolve, reject) {
return reject(new Parse.Error(Parse.Error.INTERNAL_SERVER_ERROR, 'LDAP group search failed'));
}
res.on('searchEntry', entry => {
if (entry.object.cn === options.groupCn) {
if (entry.pojo.attributes.find(obj => obj.type === 'cn').values.includes(options.groupCn)) {
found = true;
client.unbind();
client.destroy();
Expand Down

0 comments on commit f5d6fc9

Please sign in to comment.