From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-Status: No, score=-4.0 required=3.0 tests=ALL_TRUSTED,BAYES_00 shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from localhost (dcvr.yhbt.net [127.0.0.1]) by dcvr.yhbt.net (Postfix) with ESMTP id 831B21F4B4 for ; Tue, 30 Mar 2021 22:29:47 +0000 (UTC) From: Eric Wong To: meta@public-inbox.org Subject: [PATCH] lei: fix IMAP auth failure handling Date: Wed, 31 Mar 2021 03:29:47 +0500 Message-Id: <20210330222947.27332-1-e@80x24.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit List-Id: We must use the $ops hashref returned by lei->workers_start, since it's modified to include extra handlers for auth failures and whatnot. Fixes: 954581b8e575966a ("lei: simplify PktOp callers") --- lib/PublicInbox/LeiImport.pm | 2 +- lib/PublicInbox/LeiTag.pm | 2 +- xt/lei-auth-fail.t | 15 +++++++-------- 3 files changed, 9 insertions(+), 10 deletions(-) diff --git a/lib/PublicInbox/LeiImport.pm b/lib/PublicInbox/LeiImport.pm index 227a2a21..dbf655b6 100644 --- a/lib/PublicInbox/LeiImport.pm +++ b/lib/PublicInbox/LeiImport.pm @@ -76,7 +76,7 @@ sub lei_import { # the main "lei import" method my $ops = { '' => [ \&import_done, $lei ] }; $lei->{auth}->op_merge($ops, $self) if $lei->{auth}; $self->{-wq_nr_workers} = $j // 1; # locked - my ($op_c, undef) = $lei->workers_start($self, 'lei_import', $j, $ops); + (my $op_c, $ops) = $lei->workers_start($self, 'lei_import', $j, $ops); $lei->{imp} = $self; net_merge_complete($self) unless $lei->{auth}; $op_c->op_wait_event($ops); diff --git a/lib/PublicInbox/LeiTag.pm b/lib/PublicInbox/LeiTag.pm index 56ac25fa..8b012b16 100644 --- a/lib/PublicInbox/LeiTag.pm +++ b/lib/PublicInbox/LeiTag.pm @@ -116,7 +116,7 @@ sub lei_tag { # the "lei tag" method my $ops = { '' => [ \&tag_done, $lei ] }; $lei->{auth}->op_merge($ops, $self) if $lei->{auth}; $self->{vmd_mod} = $vmd_mod; - my ($op_c, undef) = $lei->workers_start($self, 'lei_tag', 1, $ops); + (my $op_c, $ops) = $lei->workers_start($self, 'lei_tag', 1, $ops); $lei->{tag} = $self; net_merge_complete($self) unless $lei->{auth}; $op_c->op_wait_event($ops); diff --git a/xt/lei-auth-fail.t b/xt/lei-auth-fail.t index 78f8466d..e352aab3 100644 --- a/xt/lei-auth-fail.t +++ b/xt/lei-auth-fail.t @@ -9,13 +9,12 @@ require_mods(qw(Mail::IMAPClient)); my $imap_fail = $ENV{TEST_LEI_IMAP_FAIL_URL} // 'imaps://AzureDiamond:Hunter2@public-inbox.org:994/INBOX'; test_lei(sub { - ok(!lei(qw(convert -o mboxrd:/dev/stdout), $imap_fail), - 'IMAP auth failure on convert'); - like($lei_err, qr!\bE:.*?imaps://.*?!sm, 'error shown'); - unlike($lei_err, qr!Hunter2!s, 'password not shown'); - is($lei_out, '', 'nothing output'); - ok(!lei(qw(import), $imap_fail), 'IMAP auth failure on import'); - like($lei_err, qr!\bE:.*?imaps://.*?!sm, 'error shown'); - unlike($lei_err, qr!Hunter2!s, 'password not shown'); + for my $pfx ([qw(convert -o mboxrd:/dev/stdout)], ['import'], + [qw(tag +L:INBOX)]) { + ok(!lei(@$pfx, $imap_fail), "IMAP auth failure on @$pfx"); + like($lei_err, qr!\bE:.*?imaps://.*?!sm, 'error shown'); + unlike($lei_err, qr!Hunter2!s, 'password not shown'); + is($lei_out, '', 'nothing output'); + } }); done_testing;