Re: System username in pg_stat_activity - Mailing list pgsql-hackers

From Julien Rouhaud
Subject Re: System username in pg_stat_activity
Date
Msg-id CAOBaU_Yp08MQOK7_k4QVyxL6sf7TURGpjX3tn1Z+WxJo2x7+GQ@mail.gmail.com
Whole thread Raw
In response to Re: System username in pg_stat_activity  (Magnus Hagander <magnus@hagander.net>)
Responses Re: System username in pg_stat_activity
List pgsql-hackers
Hi,

On Thu, Jan 18, 2024 at 11:01 PM Magnus Hagander <magnus@hagander.net> wrote:
>
> I did. Here it is, and also including that suggested docs fix as well
> as a rebase on current master.

+    if (MyClientConnectionInfo.authn_id)
+        strlcpy(lbeentry.st_auth_identity,
MyClientConnectionInfo.authn_id, NAMEDATALEN);
+    else
+        MemSet(&lbeentry.st_auth_identity, 0,
sizeof(lbeentry.st_auth_identity));

Should we use pg_mbcliplen() here?  I don't think there's any strong
guarantee that no multibyte character can be used.  I also agree with
the nearby comment about MemSet being overkill.

+       value as the identity part in <xref linkend="system-user" />, or NULL
I was looking at
https://www.postgresql.org/docs/current/auth-username-maps.html and
noticed that this page is switching between system-user and
system-username.  Should we clean that up while at it?



pgsql-hackers by date:

Previous
From: torikoshia
Date:
Subject: Re: POC PATCH: copy from ... exceptions to: (was Re: VLDB Features)
Next
From: Jakub Wartak
Date:
Subject: Re: heavily contended lwlocks with long wait queues scale badly