Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Connection URL support for Cluster constructor #1519

Open
onichandame opened this issue Mar 4, 2022 · 2 comments · May be fixed by #1794
Open

Connection URL support for Cluster constructor #1519

onichandame opened this issue Mar 4, 2022 · 2 comments · May be fixed by #1794
Labels
discussion enhancement

Comments

@onichandame
Copy link

onichandame commented Mar 4, 2022 •
edited
Loading

Problem

Currently the Cluster constructor cannot take a url string as input. Therefore the user has to manually parse a url string obtained from the runtime environment into a structured argument which the ioredis.Cluster understands.

Desired Behaviour

It would save a lot of repetitive and error-prone work if the library itself takes care of the parsing.
@luin
Copy link
Collaborator

luin commented Mar 16, 2022

The only blocker I can think of for this is to find a URI spec for multi-endpoint.

@onichandame
Copy link
Author

@luin I believe a single endpoint url can be used to access a cluster. Multi-endpoint url spec and url parsing for cluster are related but not entirely overlapping.

@steprescott steprescott linked a pull request Aug 10, 2023 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
discussion enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Connection URL support for Cluster constructor steprescott/ioredis
2 participants
@luin @onichandame