fix: workaround to fix PGHOST-related defaults picked by Postgrex

This is a temporary workaround to override any PGHOST-related value that might
inadvertently set by default

We can revert this once a new Postgrex version >0.20.0 is published

See elixir-ecto/postgrex#742 for more information

Author: davoclavo

lib/sequin/databases/postgres_database.ex

@@ -186,6 +186,12 @@ defmodule Sequin.Databases.PostgresDatabase do
       |> Enum.to_list()
       |> Keyword.put_new(:connect_timeout, @default_connect_timeout)
       |> Keyword.put(:types, Sequin.Postgres.PostgrexTypes)
+      # Temporary workaround to resolve Postgrex endpoint resolving issues if PGHOST is set to a socket or socket directory
+      # keep this until a new Postgrex version (>0.20.0) is published including this commit
+      # https://github.com/elixir-ecto/postgrex/commit/412b55567b6f0f3feb587e38466fcab047581c0f
+      |> Keyword.put(:socket, nil)
+      |> Keyword.put(:socket_dir, nil)
+      |> Keyword.put(:endpoints, nil)
 
     # TODO: Remove this when we have CA certs for the cloud providers
     # We likely need a bundle that covers many different database providers

test/sequin/databases_test.exs

@@ -106,6 +106,17 @@ defmodule Sequin.DatabasesTest do
       assert {:error, _} = Databases.tables(db)
       assert {:error, _} = Databases.update_tables(db)
     end
+
+    @tag capture_log: true
+    test "we are able connect to the database even if PGHOST is set" do
+      System.put_env("PGHOST", "/something")
+      on_exit(fn -> System.delete_env("PGHOST") end)
+
+      db = DatabasesFactory.insert_configured_postgres_database!()
+
+      assert {:ok, _} = Databases.tables(db)
+      assert {:ok, _} = Databases.update_tables(db)
+    end
   end
 
   describe "to_postgrex_opts/1" do