Content-Type: text/plain;
	charset=utf-8
Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3776.700.51.11.12\))
Subject: Re: Fix OAuth validator docs for error_detail on internal errors
From: Daniel Gustafsson <daniel@yesql.se>
In-Reply-To: <0281836A-F5FF-41A5-9EE1-656C1FAAC6B2@gmail.com>
Date: Thu, 4 Jun 2026 22:19:30 +0200
Cc: Postgres hackers <pgsql-hackers@lists.postgresql.org>,
 Jacob Champion <jacob.champion@enterprisedb.com>
Content-Transfer-Encoding: quoted-printable
Message-Id: <D9F1340C-22C2-43A5-B790-5D5EB379394A@yesql.se>
References: <0281836A-F5FF-41A5-9EE1-656C1FAAC6B2@gmail.com>
To: Chao Li <li.evan.chao@gmail.com>
Archived-At: 
 <https://www.postgresql.org/message-id/D9F1340C-22C2-43A5-B790-5D5EB379394A%40yesql.se>
Precedence: bulk

> On 4 Jun 2026, at 14:33, Chao Li <li.evan.chao@gmail.com> wrote:

> =E2=80=9CAny result parameters are ignored=E2=80=9D is no longer =
accurate; it should be something like =E2=80=9Cany result parameters =
except result->error_detail are ignored=E2=80=9D. This patch just makes =
that tiny doc fix.

That's true, but error_detail is explained in detail in the next =
paragraph so
I'm not sure this change is needed.

Another thing we don't explicitly document which seems more interesting =
is that
authn_id is used even in case of failure if log_connections is enabled.  =
Maybe
that deserves a mention?

--
Daniel Gustafsson