Fix instable 006_login_trigger.pl test
authorAlexander Korotkov <[email protected]>
Sun, 29 Oct 2023 22:23:53 +0000 (01:23 +0300)
committerAlexander Korotkov <[email protected]>
Sun, 29 Oct 2023 22:28:18 +0000 (01:28 +0300)
Handling of login trigger FATAL error could cause a timing-dependant panic of
IPC::Run.  This commit excludes checks which involves handling of such errors.

Reported-by: Tom Lane
Discussion: https://postgr.es/m/2268825.1698618066%40sss.pgh.pa.us

src/test/authentication/t/006_login_trigger.pl

index 24beb0a0b2da9c1fd49fc229468412a36665312e..3f1a8a7c457c99166b201d2efbd286e305945317 100644 (file)
@@ -143,27 +143,16 @@ psql_command(
        log_exact => '2',
        err_like => [qr/You are welcome/]);
 
-# Try to log as allowed Alice and disallowed Mallory (two times)
+# Try to login as allowed Alice.  We don't check the Mallroy login, because
+# FATAL error could cause a timing-dependant panic of IPC::Run.
 psql_command(
        $node, 'SELECT 1;', 0, 'try regress_alice',
        connstr => 'user=regress_alice',
        log_exact => '1',
        err_like => [qr/You are welcome/],
        err_unlike => [qr/You are NOT welcome/]);
-psql_command(
-       $node, 'SELECT 1;', 2, 'try regress_mallory',
-       connstr => 'user=regress_mallory',
-       log_exact => '',
-       err_like => [qr/You are NOT welcome/],
-       err_unlike => [qr/You are welcome/]);
-psql_command(
-       $node, 'SELECT 1;', 2, 'try regress_mallory',
-       connstr => 'user=regress_mallory',
-       log_exact => '',
-       err_like => [qr/You are NOT welcome/],
-       err_unlike => [qr/You are welcome/]);
 
-# Check that Alice's login record is here, while the Mallory's one is not
+# Check that Alice's login record is here
 psql_command(
        $node, 'SELECT * FROM user_logins;', 0, 'select *',
        log_like => [qr/3\|regress_alice/],