lei_mail_sync: reject IMAP URLs w/o UIDVALIDITY

It's inappropriate to store sync information without
UIDVALIDITY, so add an assertion to prevent it.

diff --git a/lib/PublicInbox/LeiMailSync.pm b/lib/PublicInbox/LeiMailSync.pm
index b298668658b650a7d269f96896e062e4e8ba835a..63076fa1c0712c45b82f44e5446ff05b8b1fe31c 100644 (file)
--- a/lib/PublicInbox/LeiMailSync.pm
+++ b/lib/PublicInbox/LeiMailSync.pm
@@ -7,6 +7,7 @@ use strict;
 use v5.10.1;
 use DBI;
 use PublicInbox::ContentHash qw(git_sha);
+use Carp ();
 
 sub dbh_new {
        my ($self, $rw) = @_;
@@ -88,6 +89,10 @@ UPDATE folders SET loc = ? WHERE fid = ?
 EOM
                        return $fid;
                }
+       } elsif ($rw && $folder =~ m!\Aimaps?://!i) {
+               require PublicInbox::URIimap;
+               PublicInbox::URIimap->new($folder)->uidvalidity //
+                       Carp::croak("BUG: $folder has no UIDVALIDITY");
        }
        return unless $rw;