-
Notifications
You must be signed in to change notification settings - Fork 19
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.
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
Support for url as a connect option? #41
Comments
@OrcaXS I did a fork of this project (https://github.com/danielfroz/tedis). Also wanted URL.... so it's implemented in this fork. Examples under .spec.ts files. |
@danielfroz PR possible? |
hi @randallb it is possible. The thing here is that tedis has problems with the parser and not supporting Streams / PUB SUB, etc... So created this fork to support my needs. Didn't receive any reply to my comment / issue. Perhaps not desired. Not sure.... |
@danielfroz Nice work! Thanks! How does one use your fork? |
Hi @sudo-kaizen . You either reference it locally (path reference locally) or create a private repository and deploy it ... something like @sudo-kaizen/tedis. But honestly not sure if you will need it . Long time I don't follow Tedis development... so maybe the author already fixed the code and implemented the requested feature. Would check the code first before testing my fork. Hope this helps |
Yup! I eventually ended up using Thanks |
It's more convenient than separate host/port options in some environments, like CI.
Since https://github.com/luin/ioredis supports
redis://
directly ootb, it would make the transition easier if tedis could support this as well.Thanks!
The text was updated successfully, but these errors were encountered: