proxysql

Commit Graph

Author	SHA1	Message	Date
Rene Cannao	49371fd647	pgsql: omit port= when port==0 (Unix socket support) Closes #5837. When pgsql_servers.port is 0 the hostname is documented to be a Unix-domain socket path. libpq rejects 'port=0' with 'invalid port number: "0"', so the field has to be omitted from the conninfo string. Add an 'if (port != 0)' guard at the three call sites that build libpq conninfo (client connect, backend kill, monitor) instead of creating a new helper header for a 3-line check. Add a regression TAP test (pgsql-socket-g1) that configures a pgsql_servers row with hostname='/var/run/postgresql-shared' and port=0, then exercises both the client connect path and the pgsql monitor. The infra uses a host-directory bind mount shared between the pgdb1 container (which creates the socket) and the ProxySQL container (which reads it), following the same pattern already used for the SSL cert directory in the same infra.	3 weeks ago

Author

SHA1

Message

Date

Rene Cannao

49371fd647

pgsql: omit port= when port==0 (Unix socket support)

Closes #5837.

When pgsql_servers.port is 0 the hostname is documented to be a
Unix-domain socket path. libpq rejects 'port=0' with 'invalid port
number: "0"', so the field has to be omitted from the conninfo
string. Add an 'if (port != 0)' guard at the three call sites that
build libpq conninfo (client connect, backend kill, monitor) instead
of creating a new helper header for a 3-line check.

Add a regression TAP test (pgsql-socket-g1) that configures a
pgsql_servers row with hostname='/var/run/postgresql-shared' and
port=0, then exercises both the client connect path and the
pgsql monitor. The infra uses a host-directory bind mount shared
between the pgdb1 container (which creates the socket) and the
ProxySQL container (which reads it), following the same pattern
already used for the SSL cert directory in the same infra.

1 Commits (fix/pgsql-unix-socket)