diff options
author | Eric Wong <e@80x24.org> | 2021-04-03 02:24:23 +0000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2021-04-03 05:08:52 +0000 |
commit | 39b7af9565f85a720e7eeb7564cfa661000cb7e9 (patch) | |
tree | 239ebb2145efc2d8ad34f550675ff2e35b341550 /lib/PublicInbox/NetReader.pm | |
parent | 385bdbda71172fc421fc4cb6018270566f3268c9 (diff) | |
download | public-inbox-39b7af9565f85a720e7eeb7564cfa661000cb7e9.tar.gz |
Leaving workers running on after auth failures is bad and messy, cleanup our process management to have consistent worker teardowns. Improve error reporting, too, instead of letting Mail::IMAPClient->exists fail due to undef.
Diffstat (limited to 'lib/PublicInbox/NetReader.pm')
-rw-r--r-- | lib/PublicInbox/NetReader.pm | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/PublicInbox/NetReader.pm b/lib/PublicInbox/NetReader.pm index 6a52b479..c269d841 100644 --- a/lib/PublicInbox/NetReader.pm +++ b/lib/PublicInbox/NetReader.pm @@ -267,7 +267,7 @@ sub imap_common_init ($;$) { $mics->{$sec} //= mic_for($self, "$sec/", $mic_args, $lei); next unless $self->isa('PublicInbox::NetWriter'); my $dst = $uri->mailbox // next; - my $mic = $mics->{$sec}; + my $mic = $mics->{$sec} // die "Unable to continue\n"; next if $mic->exists($dst); # already exists $mic->create($dst) or die "CREATE $dst failed <$uri>: $@"; } |