| From: | Alexander Lakhin <exclusion(at)gmail(dot)com> |
|---|---|
| To: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
| Cc: | Jacob Champion <jacob(dot)champion(at)enterprisedb(dot)com>, Jurij Grigorev <ju(dot)grigorev(at)ftdata(dot)ru>, pgsql-bugs(at)lists(dot)postgresql(dot)org |
| Subject: | Re: Postmaster crashes on SIGHUP when oauth_validator_libraries holds only whitespace |
| Date: | 2026-09-30 18:00:00 |
| Message-ID: | a37a3cea-5bd8-4385-a1d4-bb89394c079c@gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-bugs |
Hello Daniel,
18.09.2026 01:24, Daniel Gustafsson wrote:
> Pushed with tiny bits of tweaking and with a backpatch down to 18. Thanks!
>
> ./daniel
The test addition coined with bca67e5a3 is not very robust,
unfortunately. Buildfarm animal serinus found a way to break it [1]
as below:
# Failed test 'oauth_validator_libraries restored'
# at /home/bf/bf-build/serinus/HEAD/pgsql/src/test/modules/oauth_validator/t/001_server.pl line 152.
# got: ' '
# expected: 'validator'
# Looks like you failed 1 test of 171.
I've managed to reproduce this locally with a sleep:
--- a/src/backend/utils/misc/guc.c
+++ b/src/backend/utils/misc/guc.c
@@ -569,6 +569,7 @@ ProcessConfigFileInternal(GucContext context, bool applySettings, int elevel)
item->name, item->value)));
}
item->applied = true;
+pg_usleep(100000);
}
else if (scres == 0)
{
It makes oauth_validator/001_server fail deterministically for me.
Could you take a look, please?
[1] https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=serinus&dt=2026-09-29%2013%3A57%3A41
Best regards,
Alexander
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Jacob Champion | 2026-09-30 18:20:57 | Re: Postmaster crashes on SIGHUP when oauth_validator_libraries holds only whitespace |
| Previous Message | Tom Lane | 2026-09-30 15:24:56 | Re: BUG #19727: pg-combinebackup fails to link |