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

From: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
To: Don Seiler <don(at)seiler(dot)us>, Stephen Frost <sfrost(at)snowman(dot)net>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: [PATCH] Include application_name in "connection authorized" log message
Date: 2018-07-30 10:20:45
Message-ID: fc7dd4fc-321e-3847-f252-29f5a0effae7@2ndquadrant.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

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.

This implementation doesn't run the application_name through
check_application_name(), so it could end up logging application_name
values that are otherwise not acceptable. I'm not sure of the best way
to fix that.

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

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2018-07-30 10:21:31 Re: BUG #15182: Canceling authentication due to timeout aka Denial of Service Attack
Previous Message Emre Hasegeli 2018-07-30 09:57:38 Re: [PATCH] Improve geometric types