update copyright headers and email addresses

[public-inbox.git] / public-inbox-mda

diff --git a/public-inbox-mda b/public-inbox-mda
index 50805da2e6ecc99b679cc69dacfea1caa6e37e21..1a9469badfecd1c179601d5d7f7cba48b718ccc6 100755 (executable)
--- a/public-inbox-mda
+++ b/public-inbox-mda
@@ -1,5 +1,5 @@
 #!/usr/bin/perl -w
-# Copyright (C) 2013, Eric Wong <normalperson@yhbt.net> and all contributors
+# Copyright (C) 2013-2015 all contributors <meta@public-inbox.org>
 # License: AGPLv3 or later (https://www.gnu.org/licenses/agpl-3.0.txt)
 use strict;
 use warnings;
@@ -26,27 +26,32 @@ my $config = PublicInbox::Config->new;
 
 my $recipient = $ENV{ORIGINAL_RECIPIENT};
 defined $recipient or die "ORIGINAL_RECIPIENT not defined in ENV\n";
-my $dst = $config->lookup($recipient);
+my $dst = $config->lookup($recipient); # first check
 defined $dst or exit(1);
 my $main_repo = $dst->{mainrepo} or exit(1);
 my $filtered; # string dest
 
-if (PublicInbox::MDA->precheck($filter, $recipient) &&
+if (PublicInbox::MDA->precheck($filter, $dst->{address}) &&
     do_spamc($filter->simple, \$filtered)) {
        # update our message with SA headers (in case our filter rejects it)
-       my $msg = Email::Simple->new($filtered);
+       my $msg = Email::MIME->new(\$filtered);
        $filtered = undef;
        $filter->simple($msg);
 
-       if (PublicInbox::Filter->run($msg)) {
+       if (PublicInbox::Filter->run($msg, $filter)) {
                # run spamc again on the HTML-free message
                if (do_spamc($msg, \$filtered)) {
-                       $msg = Email::MIME->new($filtered);
+                       $msg = Email::MIME->new(\$filtered);
                        PublicInbox::MDA->set_list_headers($msg, $dst);
                        $filter->simple($msg);
 
                        my ($name, $email, $date) =
                                        PublicInbox::MDA->author_info($msg);
+
+                       END {
+                               search_index_sync($main_repo) if ($? == 0);
+                       };
+
                        local $ENV{GIT_AUTHOR_NAME} = $name;
                        local $ENV{GIT_AUTHOR_EMAIL} = $email;
                        local $ENV{GIT_AUTHOR_DATE} = $date;
@@ -56,6 +61,14 @@ if (PublicInbox::MDA->precheck($filter, $recipient) &&
                        $filter->pipe(PublicInbox::MDA->cmd, $main_repo);
                }
        }
+} else {
+       # Ensure emergency spam gets spamassassin headers.
+       # This makes it easier to prioritize obvious spam from less obvious
+       if (defined($filtered) && $filtered ne '') {
+               my $drop = Email::MIME->new(\$filtered);
+               $filtered = undef;
+               $filter->simple($drop);
+       }
 }
 exit 0; # goes to emergency
 
@@ -69,5 +82,13 @@ sub do_spamc {
                run([qw/spamc -E --headers/], \$orig, $out);
        };
 
-       return ($@ || $? || !defined($$out) || length($$out) == 0) ? 0 : 1;
+       return ($@ || $? || !defined($$out) || $$out eq '') ? 0 : 1;
+}
+
+sub search_index_sync {
+       my ($git_dir) = @_;
+       eval {
+               require PublicInbox::SearchIdx;
+               PublicInbox::SearchIdx->new($git_dir, 2)->index_sync;
+       };
 }