about summary refs log tree commit homepage
path: root/lib/PublicInbox/Over.pm
diff options
authorEric Wong <e@80x24.org>2020-12-07 07:40:49 +0000
committerEric Wong <e@80x24.org>2020-12-08 07:19:28 +0000
commitf0958e5f609c20070e6d6b33c8509989e2f237e9 (patch)
treec48fca67dc45e11af5e56a227d6615caeaf5f41f /lib/PublicInbox/Over.pm
parenta0c41f3d5e9b40bfbb066fa7f0e2e530818ccc0a (diff)
While "public-inbox-extindex --gc" invocations try to ensure
proper ordering, it is still possible for users to change
the `inboxes' tables via sqlite3(1) or similar means.  So
show a "missing://ibx_id=$ibx_id" placeholder to avoid undefined
variable warnings.

URLs such as "imaps://..." will eventually be supported as
eidx_keys, so having a URL-like "missing://" as a placeholder
probably makes sense.
Diffstat (limited to 'lib/PublicInbox/Over.pm')
1 files changed, 1 insertions, 0 deletions
diff --git a/lib/PublicInbox/Over.pm b/lib/PublicInbox/Over.pm
index f34e7fc1..51284601 100644
--- a/lib/PublicInbox/Over.pm
+++ b/lib/PublicInbox/Over.pm
@@ -275,6 +275,7 @@ SELECT eidx_key FROM inboxes WHERE ibx_id = ?
                 my $r = $_;
                 my $eidx_key = $eidx_key_sth->fetchrow_array;
+                $eidx_key //= "missing://ibx_id=$r->[0]";
                 "$eidx_key:$r->[1]:".unpack('H*', $r->[2]);
         } @$rows ];