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);