spray-session icon indicating copy to clipboard operation
spray-session copied to clipboard

Added call to redis AUTH command on RedisSessionManger construction

Open byrnedo opened this issue 9 years ago • 2 comments

For cases where redis is behind a password, I added an auth call in the constructor, maybe there's a smarter way to do this without trying the auth straight away?

byrnedo avatar May 21 '15 20:05 byrnedo

There doesn't seem to be much activity on this issue in scala-redis-nb, I think I will integrate your code as is until a better solution exists. Do you want to do some more polishing or is it in a good shape for you?

satabin avatar Jun 12 '15 09:06 satabin

I switched to password-less redis instance to avoid this, so I wouldn't merge this code since I haven't been using it.

I just commented again on scala-redis-nb asking for his input on using the auth command.

On 2015-06-12 11:40, Lucas Satabin wrote:

There doesn't seem to be much activity on this issue in scala-redis-nb, I think I will integrate your code as is until a better solution exists. Do you want to do some more polishing or is it in a good shape for you?

— Reply to this email directly or view it on GitHub https://github.com/gnieh/spray-session/pull/3#issuecomment-111430074.

byrnedo avatar Jun 12 '15 09:06 byrnedo