Skip to content
This repository was archived by the owner on May 29, 2020. It is now read-only.

Comments

Add password support for redis#9

Open
mcescalante wants to merge 2 commits intoSerpentAI:masterfrom
mcescalante:master
Open

Add password support for redis#9
mcescalante wants to merge 2 commits intoSerpentAI:masterfrom
mcescalante:master

Conversation

@mcescalante
Copy link

@mcescalante mcescalante commented Apr 9, 2018

This commit adds password support to redis by adding some expected arguments and adding arguments to StrictRedis which supports the password option.

Some rationale behind this:

  • This is almost always going to be required when deploying to an environment such as Heroku or anywhere that uses containers; there are most often passwords since it's for the security of inter-container communication.
  • Even if not using containers, and you have a Redis server, there's most likely a password set on it.
  • Many other production environments use a password even if their Redis installation is on the same VM.

I'd be happy to make any additional adjustments here. I've tested this in an application and it connects and runs successfully with no password on localhost, and with a password set on my production environment.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant