taoensso / carmine

Redis client + message queue for Clojure
https://www.taoensso.com/carmine
Eclipse Public License 1.0
1.16k stars 131 forks source link

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

Closed Skiggz closed 1 year ago

Skiggz commented 1 year ago

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 commented 1 year ago

@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 commented 1 year ago

Closing, this will addressed in forthcoming release 👍