Possibility to add a redis password #1047

Closed
opened 2026-02-04 23:34:26 +03:00 by OVERLORD · 3 comments
Owner

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:

  1. Change drivers to redis
  2. Look for the place to put the password

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):

  • Exact BookStack Version (Found in settings): v0.25.1
  • PHP Version: 7.2
  • Hosting Method (Nginx/Apache/Docker): apache in a docker container behind nginx reverse proxy

Additional context
Hope I didn't miss something

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](https://github.com/BookStackApp/BookStack/blob/master/config/database.php) **Steps To Reproduce** Steps to reproduce the behavior: 1. Change drivers to redis 2. Look for the place to put the password **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):** - Exact BookStack Version (Found in settings): v0.25.1 - PHP Version: 7.2 - Hosting Method (Nginx/Apache/Docker): apache in a docker container behind nginx reverse proxy **Additional context** Hope I didn't miss something
OVERLORD added the 🛠️ Enhancement📖 Docs Update🏭 Back-End labels 2026-02-04 23:34:26 +03:00
Author
Owner

@ssddanbrown commented on GitHub (Feb 20, 2019):

Hope I didn't miss something

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.

@ssddanbrown commented on GitHub (Feb 20, 2019): > Hope I didn't miss something 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.
Author
Owner

@msbt commented on GitHub (Feb 20, 2019):

Great, thanks!

@msbt commented on GitHub (Feb 20, 2019): Great, thanks!
Author
Owner

@ssddanbrown commented on GitHub (Mar 9, 2019):

The new format will be HOST:PORT:DATABASE:PASSWORD, Where :PASSWORD is 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_CLUSTER env option has now been removed.

@ssddanbrown commented on GitHub (Mar 9, 2019): The new format will be `HOST:PORT:DATABASE:PASSWORD`, Where `:PASSWORD` is 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_CLUSTER` env option has now been removed.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#1047