X-Git-Url: http://www.git.stargrave.org/?a=blobdiff_plain;f=script%2Fpublic-inbox-mda;h=54d0af01414a64c9458a7cf9d3005143bc5f9616;hb=a479b45117ed69d9311770fa39e6676d38f9cab2;hp=dca8a0ea0e8aa64f859676773e736d4cae77a3a1;hpb=74a3206babe0572a1494500d21267a31873af7b0;p=public-inbox.git diff --git a/script/public-inbox-mda b/script/public-inbox-mda index dca8a0ea..54d0af01 100755 --- a/script/public-inbox-mda +++ b/script/public-inbox-mda @@ -1,5 +1,5 @@ #!/usr/bin/perl -w -# Copyright (C) 2013-2019 all contributors +# Copyright (C) 2013-2020 all contributors # License: AGPL-3.0+ # # Mail delivery agent for public-inbox, run from your MTA upon mail delivery @@ -9,11 +9,11 @@ my $usage = 'public-inbox-mda [OPTIONS] < rfc2822_message'; my $precheck = grep(/\A--no-precheck\z/, @ARGV) ? 0 : 1; my ($ems, $emm); -sub do_exit { +my $do_exit = sub { my ($code) = shift; $emm = $ems = undef; # trigger DESTROY exit $code; -} +}; use Email::Simple; use PublicInbox::MIME; @@ -29,7 +29,7 @@ use PublicInbox::Spamcheck; # in case there's bugs in our code or user error. my $emergency = $ENV{PI_EMERGENCY} || "$ENV{HOME}/.public-inbox/emergency/"; $ems = PublicInbox::Emergency->new($emergency); -my $str = eval { local $/; }; +my $str = do { local $/; }; $str =~ s/\A[\r\n]*From [^\r\n]*\r?\n//s; $ems->prepare(\$str); my $simple = Email::Simple->new(\$str); @@ -48,7 +48,7 @@ if (!scalar(@$dests)) { if (!scalar(@$dests) && !defined($recipient)) { die "ORIGINAL_RECIPIENT not defined in ENV\n"; } - scalar(@$dests) or do_exit(67); # EX_NOUSER 5.1.1 user unknown + scalar(@$dests) or $do_exit->(67); # EX_NOUSER 5.1.1 user unknown } my $err; @@ -67,7 +67,7 @@ my $err; } } @$dests; -do_exit(67) if $err && scalar(@$dests) == 0; +$do_exit->(67) if $err && scalar(@$dests) == 0; $simple = undef; my $spam_ok; @@ -84,7 +84,7 @@ if ($spamc) { my $fh = $emm->fh; read($fh, $str, -s $fh); } -do_exit(0) unless $spam_ok; +$do_exit->(0) unless $spam_ok; # -mda defaults to the strict base filter which we won't use anywhere else sub mda_filter_adjust ($) { @@ -131,4 +131,4 @@ if (scalar(@rejects) && scalar(@rejects) == scalar(@$dests)) { die join("\n", @rejects, ''); } -do_exit(0); +$do_exit->(0);