Closed pyr0hu closed 4 months ago
When using secure Redis connections with TLS, Reverb will only construct a TCP connection and could not re-use the existing TLS connection.
rediss:// is the secure scheme when using TLS (See: https://www.iana.org/assignments/uri-schemes/prov/redis and also clue/reactphp-redis repository)
rediss://
clue/reactphp-redis
This PR checks if the Redis connection has a scheme option with tls as value (as per Laravel's documentation) and add's the s to the connection URL.
scheme
tls
s
PS: the $protocol variable name might not be the best choice so I'm open for suggestions on that.
$protocol
When using secure Redis connections with TLS, Reverb will only construct a TCP connection and could not re-use the existing TLS connection.
rediss://is the secure scheme when using TLS (See: https://www.iana.org/assignments/uri-schemes/prov/redis and alsoclue/reactphp-redisrepository)This PR checks if the Redis connection has a
schemeoption withtlsas value (as per Laravel's documentation) and add's thesto the connection URL.PS: the
$protocolvariable name might not be the best choice so I'm open for suggestions on that.