Fix

.github/workflows/main.yml

@@ -9,43 +9,42 @@ on:
 jobs:
   test:
     name: Test (Elixir ${{matrix.elixir}} | Erlang/OTP ${{matrix.otp}})
-    runs-on: ubuntu-20.04
+    runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: false
       matrix:
         include:
           - otp: "27.0"
-            elixir: "1.17.0"
-            version-type: strict
+            elixir: "1.17"
+            os: ubuntu-latest
             dialyzer: true
             lint: true
 
           - otp: "25.3"
             elixir: "1.14.3"
+            os: ubuntu-20.04
             coverage: true
-            version-type: loose
 
           - otp: "23.0"
             elixir: "1.12"
-            version-type: loose
+            os: ubuntu-20.04
 
     env:
-      GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}
+      GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
       MIX_ENV: test
 
     steps:
       - name: Clone the repository
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
 
       - name: Start Docker
         run: docker compose up --detach
 
       - name: Install OTP and Elixir
         uses: erlef/setup-beam@v1
         with:
-          otp-version: ${{matrix.otp}}
-          elixir-version: ${{matrix.elixir}}
-          version-type: ${{matrix.version-type}}
+          otp-version: ${{ matrix.otp }}
+          elixir-version: ${{ matrix.elixir }}
 
       - name: Cache dependencies
         id: cache-deps

lib/redix/connector.ex

@@ -137,15 +137,16 @@ defmodule Redix.Connector do
                Logger.debug(fn ->
                  "Sentinel reported #{sentinel_opts[:role]}: #{server_host}:#{server_port}"
                end),
-             {:ok, server_socket, _address} <-
+             server_host = string_address_to_erlang(server_host),
+             {:ok, server_socket, address} <-
                connect_directly(
-                 String.to_charlist(server_host),
+                 server_host,
                  String.to_integer(server_port),
                  opts
                ),
              :ok <- verify_server_role(server_socket, opts, sentinel_opts) do
           :ok = transport.close(sent_socket)
-          {:ok, server_socket, "#{server_host}:#{server_port}"}
+          {:ok, server_socket, address}
         else
           {cause, reason} when cause in [:error, :stop] ->
             :telemetry.execute([:redix, :failed_connection], %{}, %{
@@ -171,6 +172,19 @@ defmodule Redix.Connector do
     end
   end
 
+  defp string_address_to_erlang(address) when is_binary(address) do
+    address = String.to_charlist(address)
+
+    case :inet.parse_address(address) do
+      {:ok, ip} -> ip
+      {:error, :einval} -> address
+    end
+  end
+
+  defp string_address_to_erlang(address) do
+    address
+  end
+
   defp connect_to_sentinel(sentinel, sentinel_opts, transport) do
     host = Keyword.fetch!(sentinel, :host)
     port = Keyword.fetch!(sentinel, :port)

lib/redix/format.ex

@@ -14,4 +14,14 @@ defmodule Redix.Format do
 
   def format_host_and_port(host, port) when is_list(host),
     do: format_host_and_port(IO.chardata_to_string(host), port)
+
+  def format_host_and_port(host, port) when is_tuple(host) do
+    case :inet.ntoa(host) do
+      {:error, :einval} ->
+        raise ArgumentError, "invalid host: #{inspect(host)}"
+
+      host ->
+        format_host_and_port(host, port)
+    end
+  end
 end

test/redix/sentinel_test.exs

@@ -49,6 +49,7 @@ defmodule Redix.SentinelTest do
     assert_receive {:redix_pubsub, ^pubsub, ^ref, :message, %{channel: "foo", payload: "hello"}}
   end
 
+  @tag :capture_log
   test "when no sentinels are reachable" do
     Process.flag(:trap_exit, true)