Postgres Pro
Downloads
Home   >   mailing lists

Re: [PATCH] Include application_name in "connection authorized" log message - Mailing list pgsql-hackers

From Don Seiler
Subject Re: [PATCH] Include application_name in "connection authorized" log message
Date
Msg-id CAHJZqBAQyd8paBitHMvMwjcFs-TjR53nBwu4VWdvgni3GqfN_Q@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH] Include application_name in "connection authorized" logmessage  (Stephen Frost <sfrost@snowman.net>)
Responses Re: [PATCH] Include application_name in "connection authorized" logmessage
List pgsql-hackers
Tree view
On Tue, Aug 7, 2018 at 8:46 AM, Stephen Frost <sfrost@snowman.net> wrote:
Greetings,

* Don Seiler (don@seiler.us) wrote:
> On Mon, Jul 30, 2018 at 5:20 AM, Peter Eisentraut <
> peter.eisentraut@2ndquadrant.com> wrote:
>
> > On 13/07/2018 20:20, Don Seiler wrote:
> > > See attached for latest revision.
> >
> > This doesn't compile with SSL enabled because there is a comma missing.
>
> Hmm I'll check this out tonight. Sorry I wasn't able to get to this until
> now.

Thanks.

Attaching a new patch fixing the missing comma. Sorry for missing this, obviously my previous test builds didn't have SSL enabled. Mea culpa.

I'll send a new one if we require a change for the check_application_name() concern. I need the git rebase/autosquash practice anyway.

Thanks!
Don.

--
Don Seiler
www.seiler.us
Attachment

pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: garbage variable in GNUmakefile.in
Next
From: Stephen Frost
Date:
Subject: Re: [PATCH] Include application_name in "connection authorized" logmessage