mirror of
https://github.com/BookStackApp/BookStack.git
synced 2026-07-16 05:33:49 +03:00
Possibility to add a redis password #1047
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @msbt on GitHub (Feb 18, 2019).
Originally assigned to: @ssddanbrown on GitHub.
Describe the bug
When using redis as cache/session driver, one might need to add a password for the redis connection, there is no variable in the config or any mention in the db file
Steps To Reproduce
Steps to reproduce the behavior:
Expected behavior
Either have a REDIS_PASSWORD variable or use something like REDIS_SERVERS=:${REDIS_PASSWORD}@${REDIS_HOST}:${REDIS_PORT}
Your Configuration (please complete the following information):
Additional context
Hope I didn't miss something
@ssddanbrown commented on GitHub (Feb 20, 2019):
No, Just don't used redis that often myself so did not know it could be commonly used with a password.
Thanks for reporting, Should be a fairly quick thing so have marked for the next minor release.
@msbt commented on GitHub (Feb 20, 2019):
Great, thanks!
@ssddanbrown commented on GitHub (Mar 9, 2019):
The new format will be
HOST:PORT:DATABASE:PASSWORD, Where:PASSWORDis optional.Just about to merge in the code updates for this, Will be in the next patch release.
Notes for release
Previously specifying multiple redis servers did not actually seem to do anything, Only the first was used. Code is now updated to auto-use a cluster if more than one server is provided.
The
REDIS_CLUSTERenv option has now been removed.