Re: Log message for GSS connection is missing once connection authorization is successful. - Mailing list pgsql-hackers

From Euler Taveira
Subject Re: Log message for GSS connection is missing once connection authorization is successful.
Date
Msg-id CAH503wB9CsJHA+YmKZe3Q--vnVu2B2T5ygW2hzAvp-5i0nzCkg@mail.gmail.com
Whole thread Raw
In response to Re: Log message for GSS connection is missing once connection authorization is successful.  (vignesh C <vignesh21@gmail.com>)
Responses Re: Log message for GSS connection is missing once connection authorization is successful.  (Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>)
List pgsql-hackers
On Fri, 30 Oct 2020 at 09:43, vignesh C <vignesh21@gmail.com> wrote:

Attached v3 patch has the change for the same.


Hi Vignesh,

+ appendStringInfo(&logmsg, "replication ");
+
+ appendStringInfo(&logmsg, "connection authorized: user=%s",
+ port->user_name);
+ if (!am_walsender)
+ appendStringInfo(&logmsg, " database=%s", port->database_name);
+
+ if (port->application_name != NULL)
+ appendStringInfo(&logmsg, " application_name=%s",
+ port->application_name);
+

Your approach breaks localization. You should use multiple errmsg.

+$node->append_conf('postgresql.conf', "logging_collector= 'on'");
+$node->append_conf('postgresql.conf', "log_connections= 'on'");

booleans don't need quotes.


--
Euler Taveira                 http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

pgsql-hackers by date:

Previous
From: vignesh C
Date:
Subject: Re: Log message for GSS connection is missing once connection authorization is successful.
Next
From: Jimmy Angelakos
Date:
Subject: [PATCH] Clarify wording for convalidated in pg_constraint.