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

Support multiple servers in the client #408

Closed
silverwind opened this issue Dec 7, 2016 · 5 comments · Fixed by #658
Closed

Support multiple servers in the client #408

silverwind opened this issue Dec 7, 2016 · 5 comments · Fixed by #658
Labels
enhancement help-wanted

Comments

@silverwind
Copy link

url could take a array of URLs and LDAP operations could then be done in parallel, with the fastest one winning. Maybe a option to do sequential connection attempts might also make sense.
@silverwind
Copy link
Author

Made a wrapper that supports multiple servers. I had to make createClient async, which makes it incompatible to merge, but maybe you have a better idea on how to go about it.

@gurisko
Copy link

gurisko commented Apr 17, 2018

Is there any reason why ldapjs wouldn't want to support multiple server URLs?

@jsumners
Copy link
Member

I'm not opposed to a PR.

@davama
Copy link

davama commented Sep 27, 2019

this would be awesome!

@davama davama mentioned this issue Sep 27, 2019
Closed
@wision wision mentioned this issue Sep 16, 2020
Merged
@jsumners
Copy link
Member

⚠️ This issue has been locked due to age. If you have encountered a recent
problem that seems to be covered by this issue, please open a new issue.

Please include a minimal reproducible example
when opening a new issue.

@ldapjs ldapjs locked as resolved and limited conversation to collaborators Mar 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement help-wanted
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Client support multiple servers
4 participants
@silverwind @jsumners @gurisko @davama