Make the accepted log grep pattern more strict in order to not pick up any log
authoranton <anton@openbsd.org>
Sat, 10 Dec 2022 12:17:46 +0000 (12:17 +0000)
committeranton <anton@openbsd.org>
Sat, 10 Dec 2022 12:17:46 +0000 (12:17 +0000)
entries from sshd causing intermittent failures.

Discussed with bluhm@ back in August this year.

regress/usr.sbin/syslogd/args-fdexhaustion-sighup.pl
regress/usr.sbin/syslogd/args-server-tcp-reconnect.pl
regress/usr.sbin/syslogd/args-server-tls-reconnect.pl
regress/usr.sbin/syslogd/args-sighup-tcp.pl
regress/usr.sbin/syslogd/args-sighup-tls.pl
regress/usr.sbin/syslogd/args-sighup.pl
regress/usr.sbin/syslogd/args-sync-tcp.pl

index 89ca9d9..4634ed7 100644 (file)
@@ -46,7 +46,7 @@ our %args = (
        loggrep => {
            get_between2loggrep(),
            qr/Signal/ => 1,
-           qr/Accepted/ => 1,
+           qr/^Accepted$/ => 1,
        },
     },
     multifile => [
index 8bcb711..f68f396 100644 (file)
@@ -42,7 +42,7 @@ our %args = (
            $self->listen();
        })},
        loggrep => {
-           qr/Accepted/ => 2,
+           qr/^Accepted$/ => 2,
            qr/syslogd\[\d+\]: loghost .* connection close/ => 1,
            qr/syslogd\[\d+\]: (connect .*|.*connection error): $errors/ => 1,
            get_between2loggrep(),
index b53a9a3..45a514a 100644 (file)
@@ -45,7 +45,7 @@ our %args = (
            $self->listen();
        })},
        loggrep => {
-           qr/Accepted/ => 2,
+           qr/^Accepted$/ => 2,
            qr/syslogd\[\d+\]: loghost .* connection close/ => 1,
            qr/syslogd\[\d+\]: .*/.
                qr/connection error: handshake failed:/ => 1,
index 512b896..7fd335a 100644 (file)
@@ -50,7 +50,7 @@ our %args = (
        loggrep => {
            get_between2loggrep(),
            qr/Signal/ => 1,
-           qr/Accepted/ => 2,
+           qr/^Accepted$/ => 2,
        },
     },
 );
index 7023912..6290a60 100644 (file)
@@ -50,7 +50,7 @@ our %args = (
        loggrep => {
            get_between2loggrep(),
            qr/Signal/ => 1,
-           qr/Accepted/ => 2,
+           qr/^Accepted$/ => 2,
        },
     },
 );
index 53bbe08..a33a1a3 100644 (file)
@@ -41,7 +41,7 @@ our %args = (
        loggrep => {
            get_between2loggrep(),
            qr/Signal/ => 1,
-           qr/Accepted/ => 1,
+           qr/^Accepted$/ => 1,
        },
     },
     check => sub {
index e3b1181..78ab894 100644 (file)
@@ -54,7 +54,7 @@ our %args = (
                or die ref($self), " shutdown write failed: $!";
        })},
        loggrep => {
-           qr/Accepted/ => 2,
+           qr/^Accepted$/ => 2,
            get_between2loggrep(),
            get_thirdlog() => 0,
        },