Re: pgsql: doc: Clarify OAuth validator authn_id logging

Michael Paquier <michael@paquier.xyz>

From: Michael Paquier <michael@paquier.xyz>
To: Erik Rijkers <er@xs4all.nl>
Cc: Daniel Gustafsson <dgustafsson@postgresql.org>, pgsql-hackers@lists.postgresql.org
Date: 2026-06-07T23:56:04Z
Lists: pgsql-hackers
On Sat, Jun 06, 2026 at 02:36:24PM +0200, Erik Rijkers wrote:
> In doc/src/sgml/oauth-validators.sgml, and if I understand this text
> correctly (always a big if), then:
> 
> "If the validator returns true and set result->authn_id then"
> 
> should be:
> "If the validator returns true and sets result->authn_id then"
>   (i.e., set -> sets)
> 
> (added Michael in CC as he seems to collect these minimal patches, to be
> committed later as a group)

In auth-oauth.c, there is a validate() path setting the authn_id, so
your suggestion sounds right.  I've grabbed that for later.  My branch
can always be overwritten by somebody else's commit preference, of
course, so if you feel that this should be fixed separately, feel free
to go ahead.
--
Michael

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →
  1. doc: Clarify OAuth validator authn_id logging