pgsql: pg_receivewal, pg_recvlogical: allow canceling initial password - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: pg_receivewal, pg_recvlogical: allow canceling initial password
Date
Msg-id E1mosHo-0000Wq-T9@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
pg_receivewal, pg_recvlogical: allow canceling initial password prompt.

Previously it was impossible to terminate these programs via control-C
while they were prompting for a password.  We can fix that trivially
for their initial password prompts, by moving setup of the SIGINT
handler from just before to just after their initial GetConnection()
calls.

This fix doesn't permit escaping out of later re-prompts, but those
should be exceedingly rare, since the user's password or the server's
authentication setup would have to have changed meanwhile.  We
considered applying a fix similar to commit 46d665bc2, but that
seemed more complicated than it'd be worth.  Moreover, this way is
back-patchable, which that wasn't.

The misbehavior exists in all supported versions, so back-patch to all.

Tom Lane and Nathan Bossart

Discussion: https://postgr.es/m/747443.1635536754@sss.pgh.pa.us

Branch
------
REL_11_STABLE

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

Modified Files
--------------
src/bin/pg_basebackup/pg_receivewal.c  | 12 ++++++++----
src/bin/pg_basebackup/pg_recvlogical.c | 22 ++++++++++++----------
2 files changed, 20 insertions(+), 14 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: Re: pgsql: Require version 0.98 of Test::More for TAP tests
Next
From: Peter Geoghegan
Date:
Subject: pgsql: Remove lazy_scan_heap parallel VACUUM comment block.