fix for localhost redis fallback logic

[wfairclough]

Fix the Redis fallback logic in ‎NewRedisClient. When ‎REDIS_URL is empty, ‎strings.Split("", ",") returns ‎[]string{""} (length 1), so the old ‎len(addrs) == 0 check never triggered the localhost fallback. This moves the empty check before the split so the fallback to ‎localhost:6379 works correctly.

Also fixes a typo in Redis default port number (‎6397 -> ‎6379).

[Copilot]

Pull request overview

Fixes Redis client initialization so that an unset REDIS_URL correctly falls back to a local standalone Redis instance, preventing the strings.Split("", ",") => []string{""} edge case from bypassing the fallback logic.

Changes:

• Read REDIS_URL into a variable and check for empty before splitting, ensuring fallback is applied when unset.
• Correct the default localhost Redis port from 6397 to 6379.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.