pg_autoctl config check: INFO confuses PID with port and vice versa

[u1sge]

FYI: Today, I used pg_auto_failover for the very first time ;-)

After creating the monitor I wanted to learn about pg_autoctl functionality and stumbled about this minor glitch when checking my configuration

pg_autoctl config check:

15:45:34 47885 INFO  Postgres setup for PGDATA "/var/db/postgres/data18" is ok, running with PID 5432 and port 47546
15:45:34 47885 INFO  Connection to local Postgres ok, using "port=5432 dbname=pg_auto_failover host=/tmp user=autoctl_node"
pgdata:                /var/db/postgres/data18
pg_ctl:                /usr/local/bin/pg_ctl
pg_version:            "18.4" (1804)
pghost:                /tmp
pgport:                5432
proxyport:             0
pid:                   47546
is in recovery:        no
Control cluster state: in production
Control Version:       1800
Catalog Version:       202506291
System Identifier:     7648645088023558424
Latest checkpoint LSN: 0/1E413F8
Postmaster status:     ready

The first line confuses PID with port and vice versa

HTH

[xinferum]

Hello.

It looks like the order of the arguments is messed up in the code here

pg_auto_failover/src/bin/pg_autoctl/cli_config.c

Lines 268 to 270 in 5f31d4f

	log_info("Postgres setup for PGDATA \"%s\" is ok, "
	"running with PID %d and port %d",
	pgSetup->pgdata, pgSetup->pidFile.port, pgSetup->pidFile.pid);

[u1sge]

I am new here, thus: do I need to provide a git patch?
Or does a developer react on this?