memcached-session-manager icon indicating copy to clipboard operation
memcached-session-manager copied to clipboard

Redis url does not support authentication password and db index

Open jewian opened this issue 9 years ago • 2 comments

I am using Redis as storage. However if authentication password enabled on Redis, I cannot find any guide to configure the Redis password.

And there are 16 DBs available (0~15, default is 0) in Redis, there is no ways to use another DB other than 0.

Could you enhance the Redis uri to support redis://<password>@<host>:<port>/<db> ?

jewian avatar Dec 14 '16 08:12 jewian

@msellinger what do you think about this?

magro avatar Dec 28 '16 11:12 magro

It is my impression that authentication is not often used with Redis, since servers are put into a firewalled network anyway. Likewise, instead of using database numbers, a key prefix can be used. That said, the requested features should not be too complicated to implement. The URL parsing needs to be enhanced. And after connecting to the Redis server, auth() and/or select() methods need to be executed. Unfortunately, at the moment I don't have enough time to do this but if someone wants to contribute a patch I can review it if it helps.

msellinger avatar Dec 28 '16 14:12 msellinger