about summary refs log tree commit homepage
path: root/lib/PublicInbox/MDA.pm
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2015-10-03 10:02:33 +0000
committerEric Wong <e@80x24.org>2015-10-03 10:03:59 +0000
commitc68a1fcd75f103d08f8c2dabf9b3a1db4239f59c (patch)
tree5b431709093a712719136e8f07f9cac46af85aee /lib/PublicInbox/MDA.pm
parent22d8fcc3d37b9ce13a5088f1d0557078c9a84062 (diff)
downloadpublic-inbox-c68a1fcd75f103d08f8c2dabf9b3a1db4239f59c.tar.gz
Xapian has this limit for terms, and there are likely no
legitimate Message-IDs (or single header lines) this long; so
there's no need to workaround this limit.
Diffstat (limited to 'lib/PublicInbox/MDA.pm')
-rw-r--r--lib/PublicInbox/MDA.pm2
1 files changed, 2 insertions, 0 deletions
diff --git a/lib/PublicInbox/MDA.pm b/lib/PublicInbox/MDA.pm
index 25d6ae55..7025fb3c 100644
--- a/lib/PublicInbox/MDA.pm
+++ b/lib/PublicInbox/MDA.pm
@@ -6,6 +6,7 @@ use warnings;
 use Email::Address;
 use Date::Parse qw(strptime);
 use constant MAX_SIZE => 1024 * 500; # same as spamc default, should be tunable
+use constant MAX_MID_SIZE => 244; # max term size - 1 in Xapian
 use constant cmd => qw/ssoma-mda -1/;
 
 # drop plus addressing for matching
@@ -20,6 +21,7 @@ sub precheck {
         my ($klass, $filter, $address) = @_;
         my $simple = $filter->simple;
         my $mid = $simple->header("Message-ID");
+        return 0 if (length($mid) > MAX_MID_SIZE);
         return 0 unless usable_str(length('<m@h>'), $mid) && $mid =~ /\@/;
         return 0 unless usable_str(length('u@h'), $filter->from);
         return 0 unless usable_str(length(':o'), $simple->header("Subject"));