Log replication connections only when log_connections is on

Previously we'd always log replication connections, with no
way to turn them off.

diff --git a/src/backend/utils/init/postinit.c b/src/backend/utils/init/postinit.c
index 76890f2fda714a391eaf0643adbb7223b9c3c36b..f3ca5a5cd6aef4cbefbe58cd40e9613e17d274ed 100644 (file)
--- a/src/backend/utils/init/postinit.c
+++ b/src/backend/utils/init/postinit.c
@@ -218,28 +218,17 @@ PerformAuthentication(Port *port)
        if (!disable_sig_alarm(true))
                elog(FATAL, "could not disable timer for authorization timeout");
 
-       /*
-        * Log connection for streaming replication even if Log_connections
-        * disabled.
-        */
-       if (am_walsender)
+       if (Log_connections)
        {
-               if (port->remote_port[0])
+               if (am_walsender)
                        ereport(LOG,
-                                       (errmsg("replication connection authorized: user=%s host=%s port=%s",
-                                                       port->user_name,
-                                                       port->remote_host,
-                                                       port->remote_port)));
+                               (errmsg("replication connection authorized: user=%s",
+                                               port->user_name)));
                else
                        ereport(LOG,
-                               (errmsg("replication connection authorized: user=%s host=%s",
-                                               port->user_name,
-                                               port->remote_host)));
+                                       (errmsg("connection authorized: user=%s database=%s",
+                                                       port->user_name, port->database_name)));
        }
-       else if (Log_connections)
-               ereport(LOG,
-                               (errmsg("connection authorized: user=%s database=%s",
-                                               port->user_name, port->database_name)));
 
        set_ps_display("startup", false);