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.

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 redis:// URIs #60

Open
pratul opened this issue Oct 26, 2020 · 1 comment
Open

Support for redis:// URIs #60

pratul opened this issue Oct 26, 2020 · 1 comment
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@pratul
Copy link

pratul commented Oct 26, 2020

If I provide a redis:// style URI, I get an exception in RedisModeURIFactory#build. I've gone through the library source and it appears that such a URI scheme isn't supported out of the box -- even though Lettuce supports it.

However, I also cannot figure out how to pass the values from a deconstructed redis:// URI to the bundle during the Dropwizard initialization step. Any pointers?

@natalie-zamani
Copy link
Member

Hey Pratul! Thanks for the issue. Without an error trace, this likely seems related to our usage of the Guava HostAndPort class, and seems like it might be a good motivator to move away from that class.

@natalie-zamani natalie-zamani added the bug Something isn't working label Nov 18, 2020
@joschi joschi added the good first issue Good for newcomers label May 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants