]> Sergey Matveev's repositories - public-inbox.git/blobdiff - script/public-inbox-mda
mda, watch: wire up List-ID header support
[public-inbox.git] / script / public-inbox-mda
index 63096fee9a8812fe13f30010437a9b89b75d5305..2655a6c5ed064aad67b97f0c575a3212cde68344 100755 (executable)
@@ -1,6 +1,6 @@
 #!/usr/bin/perl -w
-# Copyright (C) 2013-2015 all contributors <meta@public-inbox.org>
-# License: AGPLv3 or later (https://www.gnu.org/licenses/agpl-3.0.txt)
+# Copyright (C) 2013-2019 all contributors <meta@public-inbox.org>
+# License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 #
 # Mail delivery agent for public-inbox, run from your MTA upon mail delivery
 use strict;
@@ -15,16 +15,13 @@ sub do_exit {
 }
 
 use Email::Simple;
-use Email::MIME;
-use Email::MIME::ContentType;
-$Email::MIME::ContentType::STRICT_PARAMS = 0; # user input is imperfect
+use PublicInbox::MIME;
 use PublicInbox::MDA;
 use PublicInbox::Config;
-use PublicInbox::Import;
-use PublicInbox::Git;
 use PublicInbox::Emergency;
 use PublicInbox::Filter::Base;
-use PublicInbox::Spawn qw(popen_rd);
+use PublicInbox::InboxWritable;
+use PublicInbox::Spamcheck;
 
 # n.b: hopefully we can setup the emergency path without bailing due to
 # user error, we really want to setup the emergency destination ASAP
@@ -32,54 +29,74 @@ use PublicInbox::Spawn qw(popen_rd);
 my $emergency = $ENV{PI_EMERGENCY} || "$ENV{HOME}/.public-inbox/emergency/";
 $ems = PublicInbox::Emergency->new($emergency);
 my $str = eval { local $/; <STDIN> };
+$str =~ s/\A[\r\n]*From [^\r\n]*\r?\n//s;
 $ems->prepare(\$str);
 my $simple = Email::Simple->new(\$str);
 my $config = PublicInbox::Config->new;
-
+my $key = 'publicinboxmda.spamcheck';
+my $default = 'PublicInbox::Spamcheck::Spamc';
+my $spamc = PublicInbox::Spamcheck::get($config, $key, $default);
+my $dst;
 my $recipient = $ENV{ORIGINAL_RECIPIENT};
-defined $recipient or die "ORIGINAL_RECIPIENT not defined in ENV\n";
-my $dst = $config->lookup($recipient); # first check
-defined $dst or do_exit(1);
-my $main_repo = $dst->{mainrepo} or do_exit(1);
+if (defined $recipient) {
+       $dst = $config->lookup($recipient); # first check
+}
+if (!defined $dst) {
+       my $list_id = $simple->header('List-Id');
+       if (defined $list_id && $list_id =~ /<[ \t]*(.+)?[ \t]*>/) {
+               $dst = $config->lookup_list_id($1);
+       }
+       if (!defined $dst && !defined $recipient) {
+               die "ORIGINAL_RECIPIENT not defined in ENV\n";
+       }
+       defined $dst or do_exit(67); # EX_NOUSER 5.1.1 user unknown
+}
+$dst->{mainrepo} or do_exit(67);
+$dst = PublicInbox::InboxWritable->new($dst);
 
 # pre-check, MDA has stricter rules than an importer might;
 do_exit(0) unless PublicInbox::MDA->precheck($simple, $dst->{address});
-
-$str = '';
-my $spam_ok = do_spamc($ems->fh, \$str);
 $simple = undef;
-$emm = PublicInbox::Emergency->new($emergency);
-$emm->prepare(\$str);
-$ems = $ems->abort;
-my $mime = Email::MIME->new(\$str);
-$str = '';
+my $spam_ok;
+if ($spamc) {
+       $str = '';
+       $spam_ok = $spamc->spamcheck($ems->fh, \$str);
+       # update the emergency dump with the new message:
+       $emm = PublicInbox::Emergency->new($emergency);
+       $emm->prepare(\$str);
+       $ems = $ems->abort;
+} else { # no spam checking configured:
+       $spam_ok = 1;
+       $emm = $ems;
+       my $fh = $emm->fh;
+       read($fh, $str, -s $fh);
+}
+
+my $mime = PublicInbox::MIME->new(\$str);
 do_exit(0) unless $spam_ok;
 
 my $fcfg = $dst->{filter} || '';
-my $filter;
-if ($fcfg eq 'scrub') { # TODO:
-       require PublicInbox::Filter::Mirror;
-       $filter = PublicInbox::Filter::Mirror->new;
-} else {
-       $filter = PublicInbox::Filter::Base->new;
+# -mda defaults to the strict base filter
+if ($fcfg eq '') {
+       $dst->{filter} = 'PublicInbox::Filter::Base';
+} elsif ($fcfg eq 'scrub') { # legacy alias, undocumented, remove?
+       $dst->{filter} = 'PublicInbox::Filter::Mirror';
 }
-
+my $filter = $dst->filter;
 my $ret = $filter->delivery($mime);
 if (ref($ret) && $ret->isa('Email::MIME')) { # filter altered message
        $mime = $ret;
 } elsif ($ret == PublicInbox::Filter::Base::IGNORE) {
        do_exit(0); # chuck it to emergency
 } elsif ($ret == PublicInbox::Filter::Base::REJECT) {
-       $! = $ret;
+       $! = 65; # EX_DATAERR 5.6.0 data format error
        die $filter->err, "\n";
 } # else { accept
+$filter = undef;
 
 PublicInbox::MDA->set_list_headers($mime, $dst);
-END { index_sync($main_repo) if $? == 0 };
-my $git = PublicInbox::Git->new($main_repo);
-my $im = PublicInbox::Import->new($git, $dst->{name}, $recipient);
+my $im = $dst->importer(0);
 if (defined $im->add($mime)) {
-       $im->done;
        $emm = $emm->abort;
 } else {
        # this message is similar to what ssoma-mda shows:
@@ -87,27 +104,6 @@ if (defined $im->add($mime)) {
                        $mime->header_obj->header_raw('Message-ID'),
                        " exists\n";
 }
-do_exit(0);
 
-# we depend on "report_safe 0" in /etc/spamassassin/*.cf with --headers
-sub do_spamc {
-       my ($in, $out) = @_;
-       my $rdr = { 0 => fileno($in) };
-       my ($fh, $pid) = popen_rd([qw/spamc -E --headers/], undef, $rdr);
-       my $r;
-       do {
-               $r = sysread($fh, $$out, 65536, length($$out));
-       } while (defined($r) && $r != 0);
-       close $fh or die "close failed: $!\n";
-       waitpid($pid, 0);
-
-       ($? || $$out eq '') ? 0 : 1;
-}
-
-sub index_sync {
-       my ($git_dir) = @_;
-       eval {
-               require PublicInbox::SearchIdx;
-               PublicInbox::SearchIdx->new($git_dir, 2)->index_sync;
-       };
-}
+$im->done;
+do_exit(0);