Re: [PATCH] Include application_name in "connection authorized" log message

From: Don Seiler <don(at)seiler(dot)us>
To: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
Cc: Stephen Frost <sfrost(at)snowman(dot)net>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: [PATCH] Include application_name in "connection authorized" log message
Date: 2018-07-02 13:12:07
Message-ID: CAHJZqBCCKutpAnXoo2Gj-w3WROBzyr9XmPiMDSmZ7qvAYNP+6g@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Jul 2, 2018 at 2:13 AM, Peter Eisentraut <
peter(dot)eisentraut(at)2ndquadrant(dot)com> wrote:

> On 21.06.18 16:21, Don Seiler wrote:
> > - (errmsg("connection
> > authorized: user=%s database=%s",
> > -
> > port->user_name, port->database_name)));
> > + (errmsg("connection
> > authorized: user=%s database=%s application=%s",
> > +
> > port->user_name, port->database_name, port->application_name)));
>
> Why is it "application" and not "application_name"?

I was trying to be consistent since we don't use "user_name" or
"database_name" as labels even though those are the variable names.

--
Don Seiler
www.seiler.us

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Fabrízio de Royes Mello 2018-07-02 13:22:50 Re: psql \df option for procedures
Previous Message Tom Lane 2018-07-02 13:00:12 Re: branches_of_interest.txt