From 83868a6028374ad25b57f9073891a7179c2fca6c Mon Sep 17 00:00:00 2001
From: Graham McIntire <graham@mcintire.me>
Date: Sat, 24 Jan 2026 17:49:57 -0600
Subject: [PATCH] fix: remove to_charlist conversion for Redix host option

Redix expects host to be a string, not a charlist. The to_charlist/1
conversion was causing FunctionClauseError in Redix.StartOptions.__validate_host__/1.

This matches how Phoenix PubSub Redis and Exq are configured, which both
use the string host directly without conversion.
---
 lib/towerops/redis_health_check.ex | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/towerops/redis_health_check.ex b/lib/towerops/redis_health_check.ex
index 5dfb549f..0f38bd3d 100644
--- a/lib/towerops/redis_health_check.ex
+++ b/lib/towerops/redis_health_check.ex
@@ -77,7 +77,7 @@ defmodule Towerops.RedisHealthCheck do
 
     # Build connection options
     conn_opts = [
-      host: to_charlist(host),
+      host: host,
       port: port,
       timeout: timeout_ms,
       socket_opts: [keepalive: true]