search

txstate-etc / ldap-async

A wrapper around ldapjs to provide promises, pooling, config by environment, and other conveniences.
MIT License
11 stars 5 forks source link

Any chance to upgrade this library to use ldapjsv3? #10

Closed tfrancois closed 11 months ago

tfrancois commented 1 year ago

I was wondering if there was any chance that this excellent library would be upgraded to take advantage of the changes in ldapjs v3 - which is a major update to v2? Thanks.

wickning1 commented 1 year ago

Yes, I’ll take a look. It seems like they have a new Filters API that I haven’t seen before, looks pretty nice.

genio commented 1 year ago

Looks like this can be closed

wickning1 commented 1 year ago

I'm working on this this week. They really changed the API in 3 so I am changing my API too and will cut a new major version.

tfrancois commented 1 year ago

Fantastic news! Thank you good sir - please keep us posted.

wickning1 commented 1 year ago

I coded everything up and put it up as PR #12 for a few days before release.

wickning1 commented 1 year ago

Just as an update, we discovered an issue with ldapjs 3 where it will not support range requests on attributes, so this is blocked pending a successful PR. We're working on that PR now.

wickning1 commented 11 months ago

ldap-async 2.0 is published

tfrancois commented 11 months ago

@wickning1 You're a rock star! Installing today - thank you sir!