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-29 09:46:19 +0000
committerEric Wong <e@80x24.org>2021-04-30 06:41:36 +0000
commit90155b72ccd38d51415f12547e39c4d65c469fe1 (patch)
tree570c61708a093f24dcba9f05d415c22a2a2cbc23 /lib/PublicInbox/NetReader.pm
parent923e09d84e8eaa612c85f6d5ec57c3742390bebc (diff)
downloadpublic-inbox-90155b72ccd38d51415f12547e39c4d65c469fe1.tar.gz
Specifying a UIDVALIDITY value allows the user to enforce
a strict match and force failure.  This necessitated changes
to NetReader to allow die() and make error reporting more
suitable for CLI usage rather than daemonized usage of -watch.
Diffstat (limited to 'lib/PublicInbox/NetReader.pm')
-rw-r--r--lib/PublicInbox/NetReader.pm7
1 files changed, 7 insertions, 0 deletions
diff --git a/lib/PublicInbox/NetReader.pm b/lib/PublicInbox/NetReader.pm
index 81d25ead..3fc37b10 100644
--- a/lib/PublicInbox/NetReader.pm
+++ b/lib/PublicInbox/NetReader.pm
@@ -407,6 +407,11 @@ sub _imap_fetch_all ($$$) {
                 return "E: $orig_uri cannot get UIDVALIDITY";
         $r_uidnext //= $mic->uidnext($mbx) //
                 return "E: $orig_uri cannot get UIDNEXT";
+        my $expect = $orig_uri->uidvalidity // $r_uidval;
+        return <<EOF if $expect != $r_uidval;
+E: $orig_uri UIDVALIDITY mismatch (got $r_uidval)
+EOF
+
         my $uri = $orig_uri->clone;
         my ($itrk, $l_uid, $l_uidval) = _itrk_last($self, $uri, $r_uidval);
         return <<EOF if $l_uidval != $r_uidval;
@@ -520,6 +525,7 @@ sub imap_each {
         } else {
                 $err = "E: <$uri> not connected: $!";
         }
+        die $err if $err && $self->{-can_die};
         warn $err if $err;
         $mic;
 }
@@ -620,6 +626,7 @@ sub nntp_each {
         } else {
                 $err = "E: <$uri> not connected: $!";
         }
+        die $err if $err && $self->{-can_die};
         warn $err if $err;
         $nn;
 }