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

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

On Tue, Aug 7, 2018 at 8:46 AM, Stephen Frost <sfrost(at)snowman(dot)net> wrote:

> Greetings,
>
> * Don Seiler (don(at)seiler(dot)us) wrote:
> > On Mon, Jul 30, 2018 at 5:20 AM, Peter Eisentraut <
> > peter(dot)eisentraut(at)2ndquadrant(dot)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 Content-Type Size
0001-Changes-to-add-application_name-to-Port-struct-so-we.patch application/octet-stream 3.3 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Stephen Frost 2018-08-07 14:20:03 Re: [PATCH] Include application_name in "connection authorized" log message
Previous Message Tom Lane 2018-08-07 14:06:28 Re: garbage variable in GNUmakefile.in