Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable support for TLS connections (predis) #457

Closed
mvehar opened this issue Aug 31, 2018 · 4 comments
Closed

Enable support for TLS connections (predis) #457

mvehar opened this issue Aug 31, 2018 · 4 comments
Labels
predis Specific to Predis support Questions about usage and features

Comments

@mvehar
Copy link

mvehar commented Aug 31, 2018

No description provided.

@curry684
Copy link
Collaborator

Does Predis even support it?

Also - doesn't it more or less defeat the entire purpose of having the fastest in-memory key-value store on earth if you're then going to waste time setting up a TLS connection to it on every request? I'm all for the generic concept of "encrypt everything" but I've always considered Redis the single exception to it, and instead lock it down by (implicit) network security instead.

@curry684 curry684 added support Questions about usage and features predis Specific to Predis labels Sep 3, 2018
@walterheck
Copy link

some environments require it for compliance reasons. AWS Elasticache redis has options for it, so it would be really good to support it here also.

it's mentioned in the main documentation page of predis also: https://github.com/nrk/predis/

@mvehar
Copy link
Author

mvehar commented Nov 17, 2018

On 2.1 branch (https://github.com/snc/SncRedisBundle/blob/2.1/DependencyInjection/Configuration/RedisDsn.php) is support already implemented. Pull request to update master branch is in progress.

@B-Galati
Copy link
Collaborator

B-Galati commented Feb 5, 2019

This feature is supported on 2.1 and 3.0.0 to be released.
Thus I am closing this issue.
Let me know if I missed something.
See #445 .

@B-Galati B-Galati closed this as completed Feb 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
predis Specific to Predis support Questions about usage and features
Projects
None yet
Development

No branches or pull requests

4 participants