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

Carmine connection seems to fail for 7.x redis - might be lack of support for SSL or "rediss:/" uri #282

Closed
Skiggz opened this issue May 26, 2023 · 2 comments

Comments

@Skiggz
Copy link

Skiggz commented May 26, 2023

URI format "rediss://" doesn't seem to be supported. Is it expected that these connections would fail? I'm trying to connect to a 7.0.x instance, whereas connecting to 6.2.x is fine.

The only thing that is odd, is that I can't connect to 7.0.x even if I don't use TLS (change the uri to "redis:/") but I'm wondering if the server is rejecting that (it's hard to tell, the error just says connection reset). Trying to eliminate possibilities and wondering if carmine simply doesn't support this.

@ptaoussanis
Copy link
Member

@Skiggz Hi Skylar, thanks for pinging about this!

"rediss://" URIs are definitely not supported, since I'd never seen them before ^^
Just added support to master now, will be included in the next release.

In the meantime (and actually in any case), I'd suggest using an explicit config map since these are a lot more flexible.

The only thing that is odd, is that I can't connect to 7.0.x even if I don't use TLS (change the uri to "redis:/")

There's no reason that Carmine should have trouble connecting to Redis 7.x.
I'd recommend playing with the explicit config options linked above. In particular, I suspect you may need to include :ssl-fn :default in your connection config map. It's possible your Redis 7.x server is rejecting non-TLS connection attempts.

If that doesn't help, it might be worth also checking if there's been any changes in your Redis server's ACL policy.

Feel free to ping if none of that was helpful. Cheers! :-)

@ptaoussanis
Copy link
Member

Closing, this will addressed in forthcoming release 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants