allow tls as scheme in DSN string for predis library

[jeff1985]

Predis allows connecting to TLS encrypted endpoints by using scheme=tls which is not allowed by current enqueue library. This is needed to connect to major cloud provider's redis services. I added the posibility to specify tls:// endpoints in DSN string for each configuration.

See predis docs for more info on TLS: https://github.com/nrk/predis

My DSN string looks as following:

ENQUEUE_DSN=tls://xxx.redis.cache.windows.net:6380?password=yyyy&ssl[verify_peer]=1&persistent=1&vendor=predis

[makasim]

Tls is to broad, I’d suggest changing it to scheme redis+tls

[jeff1985]

Hi @makasim ,

just tested: redis+tls throws exception in Predis library:

Unknown connection scheme: 'redis+tls'

another possibility would be rediss which is alias of tls. I just tested and it just works fine.

Agree?

[makasim]

Agree

[jeff1985]

OK @makasim, I changed it to rediss.

Please accept the MR!

[makasim]

Could you please add some tests to RedisConnectionFactoryConfigTest?

[jeff1985]

Hi @makasim,

I added tests to RedisConnectionFactoryConfigTest as you requested!

[jeff1985]

Hi @makasim

were you able to finish your review?

[jeff1985]

Thanks for that!