[PR #2609] [MERGED] fix: fatal with clear message when keyvalue type is redis but redis is not enabled #5765

Closed
opened 2026-04-16 13:51:36 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/go-vikunja/vikunja/pull/2609
Author: @kolaente
Created: 4/12/2026
Status: Merged
Merged: 4/12/2026
Merged by: @kolaente

Base: mainHead: claude/fix-vikunja-2608-eHoKb


📝 Commits (1)

  • 988f99c fix: fatal with clear message when keyvalue type is redis but redis is not enabled

📊 Changes

1 file changed (+4 additions, -0 deletions)

View changed files

📝 pkg/modules/keyvalue/keyvalue.go (+4 -0)

📄 Description

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 go-vikunja/vikunja#2608

https://claude.ai/code/session_01TRuPTGYDQjxqHRFWQaJGvy


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/go-vikunja/vikunja/pull/2609 **Author:** [@kolaente](https://github.com/kolaente) **Created:** 4/12/2026 **Status:** ✅ Merged **Merged:** 4/12/2026 **Merged by:** [@kolaente](https://github.com/kolaente) **Base:** `main` ← **Head:** `claude/fix-vikunja-2608-eHoKb` --- ### 📝 Commits (1) - [`988f99c`](https://github.com/go-vikunja/vikunja/commit/988f99cf1c5377ea3f55ef6e42718ecbee10fc04) fix: fatal with clear message when keyvalue type is redis but redis is not enabled ### 📊 Changes **1 file changed** (+4 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `pkg/modules/keyvalue/keyvalue.go` (+4 -0) </details> ### 📄 Description 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 go-vikunja/vikunja#2608 https://claude.ai/code/session_01TRuPTGYDQjxqHRFWQaJGvy --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-16 13:51:36 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/vikunja#5765