pgsql: oauth_validator: Remove racing GUC check

From: Jacob Champion <jchampion(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: oauth_validator: Remove racing GUC check
Date: 2026-10-08 18:02:22
Message-ID: E1xEsRi-00000000qLo-0hDP@gemulon.postgresql.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

oauth_validator: Remove racing GUC check

Follow-up to bca67e5a3. The test added there changed the value of
oauth_validator_libraries, waited for the postmaster to apply it, and
verified the change on an existing background connection. But there's no
guarantee that the SIGHUP will have arrived and completed its work on
existing backends by that time, so the final test flaked.

We only care about the postmaster's state here, since we're testing the
GUC's effects on newly forked backends, so remove the background check
entirely.

Per buildfarm member serinus.

Reported-by: Alexander Lakhin <exclusion(at)gmail(dot)com>
Discussion: https://postgr.es/m/a37a3cea-5bd8-4385-a1d4-bb89394c079c%40gmail.com
Backpatch-through: 18

Branch
------
REL_18_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/af28fc13e20db49d559c00a8156753090c5516fd

Modified Files
--------------
src/test/modules/oauth_validator/t/001_server.pl | 2 --
1 file changed, 2 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Robert Haas 2026-10-08 18:17:05 pgsql: Add child_append_relid_sets to ElidedNode; use in pg_plan_advice
Previous Message Andrew Dunstan 2026-10-08 17:41:25 pgsql: COPY TO FORMAT JSON: use the query's column names as keys