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

Issue with guide example #347

Closed
jorgen-17 opened this issue Feb 11, 2016 · 1 comment
Closed

Issue with guide example #347

jorgen-17 opened this issue Feb 11, 2016 · 1 comment

Comments

@jorgen-17
Copy link

Hey, loving ldapjs so far, its definately been making learning and setting up ldap a lot more clearer than using slapd directly. Anyways just noticed there is a mistake in the examples located at http://ldapjs.org/guide.html. The examples look for a cn property in rdns[0]
i.e.

req.dn.rdns[0].cn

but they should be looking for it in rdns[0].attrs
i.e.

req.dn.rdns[0].attrs.cn

Other than this the documentation is a huge improvement over the slapd docs I've seen out there. Thanks for putting this together!

@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.
Projects
None yet
Development

No branches or pull requests

2 participants