mirror of
https://git.postgresql.org/git/postgresql.git
synced 2024-09-04 01:27:20 +02:00
Fix potential rare failure in the kerberos TAP tests
Instead of writing a query to psql's stdin, which can cause a failure where psql exits before writing, reporting a write failure with a broken pipe, this changes the logic to use -c. This was not seen in the buildfarm as no animals with a sensitive environment are running the kerberos tests, but let's be safe. HEAD is able to handle the situation as of6d41dd0
for all the test suites doing connection checks.f44b9b6
has fixed the same problem for the LDAP tests. Discussion: https://postgr.es/m/YGu7ceWAiSNQDgH5@paquier.xyz Backpatch-through: 11
This commit is contained in:
parent
e7bcfd717e
commit
b55619948e
@ -177,13 +177,15 @@ sub test_access
|
|||||||
# need to connect over TCP/IP for Kerberos
|
# need to connect over TCP/IP for Kerberos
|
||||||
my ($res, $stdoutres, $stderrres) = $node->psql(
|
my ($res, $stdoutres, $stderrres) = $node->psql(
|
||||||
'postgres',
|
'postgres',
|
||||||
"$query",
|
undef,
|
||||||
extra_params => [
|
extra_params => [
|
||||||
'-XAtd',
|
'-XAtd',
|
||||||
$node->connstr('postgres')
|
$node->connstr('postgres')
|
||||||
. " host=$host hostaddr=$hostaddr $gssencmode",
|
. " host=$host hostaddr=$hostaddr $gssencmode",
|
||||||
'-U',
|
'-U',
|
||||||
$role
|
$role,
|
||||||
|
'-c',
|
||||||
|
$query
|
||||||
]);
|
]);
|
||||||
|
|
||||||
# If we get a query result back, it should be true.
|
# If we get a query result back, it should be true.
|
||||||
|
Loading…
Reference in New Issue
Block a user