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

Commit f93234e

Browse files
authored
Merge branch 'next' into next
2 parents d7d2e99 + 1395ce8 commit f93234e

File tree

5 files changed

+39
-4
lines changed

5 files changed

+39
-4
lines changed

.github/workflows/integration.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,9 +31,9 @@ jobs:
3131
env:
3232
GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}
3333
- name: Pull Docker image
34-
run: docker pull "docker.pkg.github.com/ldapjs/docker-test-openldap/openldap:1.0"
34+
run: docker pull "docker.pkg.github.com/ldapjs/docker-test-openldap/openldap:latest"
3535
- name: Start OpenLDAP service
36-
run: docker run -it -d --name openldap -p 389:389 -p 636:636 docker.pkg.github.com/ldapjs/docker-test-openldap/openldap:1.0
36+
run: docker run -it -d --name openldap -p 389:389 -p 636:636 docker.pkg.github.com/ldapjs/docker-test-openldap/openldap:latest
3737

3838
- name: Install Packages
3939
run: npm install

.taprc

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
esm: false
2+
jsx: false
3+
ts: false
24

35
files:
46
- 'test/**/*.test.js'

docker-compose.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ version: '3'
22

33
services:
44
openldap:
5-
image: docker.pkg.github.com/ldapjs/docker-test-openldap/openldap:1.0
5+
image: docker.pkg.github.com/ldapjs/docker-test-openldap/openldap:latest
66
ports:
77
- 389:389
88
- 636:636

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@
3030
"husky": "^3.0.4",
3131
"snazzy": "^8.0.0",
3232
"standard": "^14.0.2",
33-
"tap": "^14.6.1",
33+
"tap": "14.10.1",
3434
"uuid": "^3.3.3"
3535
},
3636
"scripts": {

test-integration/client/issues.test.js

Lines changed: 33 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,3 +56,36 @@ tap.test('whois works correctly (issue #370)', t => {
5656
})
5757
})
5858
})
59+
60+
tap.test('can access large groups (issue #582)', t => {
61+
const client = ldapjs.createClient({ url: baseURL })
62+
client.bind('cn=admin,dc=planetexpress,dc=com ', 'GoodNewsEveryone', (err) => {
63+
t.error(err)
64+
const searchOpts = {
65+
scope: 'sub',
66+
filter: '(&(objectClass=group)(cn=large_group))'
67+
}
68+
client.search('ou=large_ou,dc=planetexpress,dc=com', searchOpts, (err, response) => {
69+
t.error(err)
70+
71+
const results = []
72+
response.on('searchEntry', (entry) => {
73+
results.push(entry)
74+
})
75+
response.on('error', t.error)
76+
response.on('end', (result) => {
77+
t.is(result.status, 0)
78+
t.is(results.length === 1, true)
79+
t.ok(results[0].attributes)
80+
81+
const memberAttr = results[0].attributes.find(a => a.type === 'member')
82+
t.ok(memberAttr)
83+
t.ok(memberAttr.vals)
84+
t.type(memberAttr.vals, Array)
85+
t.is(memberAttr.vals.length, 2000)
86+
87+
client.unbind(t.end)
88+
})
89+
})
90+
})
91+
})

0 commit comments

Comments
 (0)