You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hi,
With the recent merging of SSL support into master in hiredis (redis/hiredis#645), it would be great to have the ability to make secure connections to redis via redux.
An initial implementation of this is available in jcohen02/redux@1c0232b but I wanted to get thoughts on this prior to submitting a PR - I'm assuming this will require a modified version of hiredis to be made available in rwinlib/hiredis but I wasn't clear at this stage whether it might present other cross-platform/dependency issues.
The text was updated successfully, but these errors were encountered:
@richfitz, @jeroen the feature/ssl branch in my fork has now been updated to use the modified API in hiredis 1.0.0.
I've updated tools/winlibs.R to pull hiredis-1.0.0 from rwinlib/hiredis and updated Makevars.win to add the necessary libraries for building with SSL support.
I had some issues on Mac OS since I was building using my own build of hiredis-1.0.0 and avoiding the brew-based scripts, this may need some further checking along with Linux builds.
Should we be looking to build with SSL support by default or is there some way to pass an option via install.packages or similar to specify whether or not to install redux with SSL support - is this necessary.
Once I've completed some further testing, I'll open a PR in preparation for getting this merged into the main repo.
Hi,
With the recent merging of SSL support into master in hiredis (redis/hiredis#645), it would be great to have the ability to make secure connections to redis via redux.
An initial implementation of this is available in jcohen02/redux@1c0232b but I wanted to get thoughts on this prior to submitting a PR - I'm assuming this will require a modified version of hiredis to be made available in rwinlib/hiredis but I wasn't clear at this stage whether it might present other cross-platform/dependency issues.
The text was updated successfully, but these errors were encountered: