about summary refs log tree commit homepage
path: root/lib/PublicInbox/NetReader.pm
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2021-04-03 02:24:23 +0000
committerEric Wong <e@80x24.org>2021-04-03 05:08:52 +0000
commit39b7af9565f85a720e7eeb7564cfa661000cb7e9 (patch)
tree239ebb2145efc2d8ad34f550675ff2e35b341550 /lib/PublicInbox/NetReader.pm
parent385bdbda71172fc421fc4cb6018270566f3268c9 (diff)
downloadpublic-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.pm2
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>: $@";
         }