RedisFeatureStore does not offer ssl_ca_certs kwarg

[cn-alexboyd]

Is your feature request related to a problem? Please describe.

I would love for the RedisFeatureStore to allow for me to specify a host, port, password, SSL, and ssl_ca_cert if applicable when trying to connect to a Redis cluster that has TLS/SSL enabled.

Describe the solution you'd like

Ideally, I could use the existing RedisFeatureStore.new_feature_store() method with a new set of kwargs instead of just the connection url.

store = Redis.new_feature_store(host=..., port=..., password=..., ssl=True, ssl_ca_certs=...)

Describe alternatives you've considered

The required redis package already supports these args, I think the adjustment will be using something other than redis.ConnectionPool.from_url during initialization of _RedisFeatureStoreCore.

[eli-darkly]

At first glance that seems like a reasonable and straightforward enhancement. It's hard for us to cover this in automated testing, so we would have to trust that the behavior of the redis package is correct if we pass it these parameters, but I'll look into implementing it in the near future.

[danie1k]

SSL-related and other Redis connection options can be easily set directly in the URL, for example:

qs = urllib.parse.urlencode({
    "ssl_cert_reqs": "required",
    "ssl_ca_certs": "/absolute/path/to/ca.crt",
    "ssl_keyfile": "/absolute/path/to/client.key",
    "ssl_certfile": "/absolute/path/to/client.crt",
})

store = Redis.new_feature_store(f'rediss://:password@host:port/0?{qs}')

[keelerm84]

Fixed in 7.5.0