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 →
-
doc: Clarify OAuth validator authn_id logging
- 4cb2e2fe0aa8 19 (unreleased) cited