Connection via TLS (rediss) broken since 2.1.4

[lucasvanlierop]

v2.1.4 introduced a new way of parsing the dsn via \Snc\RedisBundle\DependencyInjection\Configuration\RedisDsn. It seems to discard the rediss:// scheme which results in an dsn with the tcp:// scheme rather than the expected tls:// scheme.

We (me and @jankramer) already took a quick look on fixing it by setting the scheme in the RedisDsn and retrieving it later on when building the dsn but we would first like to hear if you have a prefered way of fixing this.

FYI: We are using the Predis client which, in contrary to PHPRedis, supports TLS

[B-Galati]

Thank you!

If the logic stays in the parseUrl method, it would be a good way of fixing it IMHO

[lucasvanlierop]

Fixed by #445