More recent Redis servers allow / require username + password combinations. This isn’t currently supported by Vikunja, but should be extremely easy to add, probably only touching a couple of files (and some documentation):
The Redis client supports the Username field:
- https:// github .com/redis/go-redis/blob/master/options.go (sorry, new users are only allowed 2 links!)
Also, while I’m here, there are a couple of other things that I noticed when setting Vikunja up on a server using non-standard ports / unusable Redis:
- If Redis is not enabled, but
keyvalue.typeis accidentally left as “redis”, the app should probably exit gracefully when the config is parsed with a useful error, rather than crashing when it can’t access Redis (despite this being completely user error). - It’s possibly worth adding separate
portoptions for both the database and Redis (whilst also allowing them to be set via the host). I only found out that this was how it was done for Postgres when I looked at the code (whereas the Redis docs specifically mention the port).
[P.S. I tried to register on the Gitea site to post an issue / create a pull request (I don’t do Github linking), but just received a message to contact the site admin and received no email.]