about summary refs log tree commit homepage
path: root/lib/PublicInbox/Search.pm
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2019-05-24 01:12:08 +0000
committerEric Wong <e@80x24.org>2019-05-24 02:58:22 +0000
commit5b05a7c7a65a25b938ebf2fdbce0d4b4fadfbd3c (patch)
treeba1839e4585ad62f90bc7c8e76bdb5c962f8aab3 /lib/PublicInbox/Search.pm
parent666f1b8f5c7c76333df4e1296c1668abf04f210f (diff)
downloadpublic-inbox-5b05a7c7a65a25b938ebf2fdbce0d4b4fadfbd3c.tar.gz
Some users (or bots :P) can trigger horrible queries which
the caller can choose to either log or ignore.  This prevents
horrible queries from ExtMsg from logging confusing "ref: "
messages when $@ is not a Perl reference.
Diffstat (limited to 'lib/PublicInbox/Search.pm')
-rw-r--r--lib/PublicInbox/Search.pm4
1 files changed, 3 insertions, 1 deletions
diff --git a/lib/PublicInbox/Search.pm b/lib/PublicInbox/Search.pm
index d861cf47..c054a874 100644
--- a/lib/PublicInbox/Search.pm
+++ b/lib/PublicInbox/Search.pm
@@ -228,7 +228,9 @@ sub retry_reopen {
                         warn "reopen try #$i on $@\n";
                         reopen($self);
                 } else {
-                        warn "ref: ", ref($@), "\n";
+                        # let caller decide how to spew, because ExtMsg queries
+                        # get wonky and trigger:
+                        # "something terrible happened at .../Xapian/Enquire.pm"
                         die;
                 }
         }