Skip to content

fix: fatal with clear message when keyvalue type is redis but redis is not enabled - #2609

Merged
kolaente merged 1 commit into
mainfrom
claude/fix-vikunja-2608-eHoKb
Apr 12, 2026
Merged

fix: fatal with clear message when keyvalue type is redis but redis is not enabled#2609
kolaente merged 1 commit into
mainfrom
claude/fix-vikunja-2608-eHoKb

Conversation

@kolaente

Copy link
Copy Markdown
Member

Instead of panicking with a nil pointer dereference when keyvalue.type
is set to "redis" but redis.enabled is false, log a fatal error with a
clear, actionable message telling the user to enable redis.

Closes #2608

https://claude.ai/code/session_01TRuPTGYDQjxqHRFWQaJGvy

…s not enabled

Instead of panicking with a nil pointer dereference when keyvalue.type
is set to "redis" but redis.enabled is false, log a fatal error with a
clear, actionable message telling the user to enable redis.

Closes #2608

https://claude.ai/code/session_01TRuPTGYDQjxqHRFWQaJGvy
@github-actions github-actions Bot added area/config config.yml, env vars, runtime flags, deployment config area/internal-code Internal refactoring, cleanup, code-quality work labels Apr 12, 2026
@kolaente
kolaente enabled auto-merge April 12, 2026 09:38
@github-actions

Copy link
Copy Markdown

Preview Deployment

Preview deployments for this PR are available at:

URL Tag Commit
https://pr-2609.preview.vikunja.dev ghcr.io/go-vikunja/vikunja:pr-2609 latest
https://sha-3e59a654b51bbd3012c8f2078e14c9ee3368405d.preview.vikunja.dev ghcr.io/go-vikunja/vikunja:sha-3e59a654b51bbd3012c8f2078e14c9ee3368405d 3e59a65

The preview environment will start automatically on first visit. Subsequent pushes to this PR will update the pr-2609 image — the preview picks up the new version on restart. The per-commit URLs point to a specific version and will not change.

Run locally with Docker
docker pull ghcr.io/go-vikunja/vikunja:pr-2609
docker run -p 3456:3456 ghcr.io/go-vikunja/vikunja:pr-2609

Last updated for commit 3e59a65

@kolaente
kolaente added this pull request to the merge queue Apr 12, 2026
Merged via the queue into main with commit 85cfadc Apr 12, 2026
38 checks passed
@kolaente
kolaente deleted the claude/fix-vikunja-2608-eHoKb branch April 12, 2026 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/config config.yml, env vars, runtime flags, deployment config area/internal-code Internal refactoring, cleanup, code-quality work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Panic with keyvalue type redis and redis not enabled

2 participants