Skip to content

feat: add redis_password support#279

Closed
carlowahlstedt wants to merge 8 commits into
signalbot-org:mainfrom
carlowahlstedt:feat/redis-password
Closed

feat: add redis_password support#279
carlowahlstedt wants to merge 8 commits into
signalbot-org:mainfrom
carlowahlstedt:feat/redis-password

Conversation

@carlowahlstedt
Copy link
Copy Markdown
Contributor

Summary

Adds optional redis_password support to RedisStorage so bots can connect to password-protected Redis instances.

Changes

  • signalbot/storage.py: RedisStorage.__init__ now accepts an optional password: str | None = None parameter, which is forwarded to redis.Redis().
  • signalbot/bot.py: Bot initialization reads redis_password from the storage config (via .get(), so it remains optional) and passes it to RedisStorage.
  • README.md: Documents the new redis_password config option under the Persistent storage section.

Usage

bot = SignalBot({
    "signal_service": "127.0.0.1:8080",
    "phone_number": "+49123456789",
    "storage": {
        "redis_host": "redis",
        "redis_port": 6379,
        "redis_password": "your_password",
    },
})

The field is optional — existing configs without redis_password continue to work unchanged (password defaults to None).

🤖 Generated with Claude Code

carlowahlstedt and others added 8 commits December 18, 2025 19:28
Co-authored-by: Era Dorta <era.dorta@proton.me>
Allow users to configure a password for Redis authentication by
adding an optional redis_password field to the storage config.

- RedisStorage.__init__ now accepts an optional password parameter
- Bot reads redis_password from config and passes it to RedisStorage
- README documents the new redis_password config option

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@carlowahlstedt
Copy link
Copy Markdown
Contributor Author

Closing in favor of a conflict-free PR based on upstream/main.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant