pgsql: Add application_name to connection authorized msg - Mailing list pgsql-committers

From Stephen Frost
Subject pgsql: Add application_name to connection authorized msg
Date
Msg-id E1g61pW-0002dS-31@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Add application_name to connection authorized msg

The connection authorized message has quite a bit of useful information
in it, but didn't include the application_name (when provided), so let's
add that as it can be very useful.

Note that at the point where we're emitting the connection authorized
message, we haven't processed GUCs, so it's not possible to get this by
using log_line_prefix (which pulls from the GUC).  There's also
something to be said for having this included in the connection
authorized message and then not needing to repeat it for every line, as
having it in log_line_prefix would do.

The GUC cleans the application name to pure-ascii, so do that here too,
but pull out the logic for cleaning up a string into its own function
in common and re-use it from those places, and check_cluster_name which
was doing the same thing.

Author: Don Seiler <don@seiler.us>
Discussion: https://postgr.es/m/CAHJZqBB_Pxv8HRfoh%2BAB4KxSQQuPVvtYCzMg7woNR3r7dfmopw%40mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/8bddc864000f56d396621d4ad0f13e8e1872ddf5

Modified Files
--------------
src/backend/postmaster/postmaster.c | 16 +++++++++++
src/backend/utils/init/postinit.c   | 54 ++++++++++++++++++++++++++-----------
src/backend/utils/misc/guc.c        | 17 +++---------
src/common/string.c                 | 34 +++++++++++++++++++++++
src/include/common/string.h         |  1 +
src/include/libpq/libpq-be.h        |  7 +++++
6 files changed, 99 insertions(+), 30 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Improve error reporting for unsupportedeffective_io_concurrency
Next
From: Tom Lane
Date:
Subject: pgsql: Create an RTE field to record the query's lock mode for eachrel