Log replication connections only when log_connections is on

mhagander · mhagander · commit 6e1726d082b5 · 2011-01-18T20:02:25.000+01:00
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
@@ -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);
 
