X-Git-Url: http://www.git.stargrave.org/?a=blobdiff_plain;f=lib%2FPublicInbox%2FMDA.pm;h=9cafda13997f68229d8f6f4f0b1dc34ce4bc3b93;hb=9bd675d33ad1e49bd2ebe12a1d216216e61380de;hp=2e6e9ec5a1e55c509f7690d5eea5edd8d5ce1bfe;hpb=23a4e44bedabe5b8b651346cabc2a870c5377a30;p=public-inbox.git
diff --git a/lib/PublicInbox/MDA.pm b/lib/PublicInbox/MDA.pm
index 2e6e9ec5..9cafda13 100644
--- a/lib/PublicInbox/MDA.pm
+++ b/lib/PublicInbox/MDA.pm
@@ -1,5 +1,5 @@
-# Copyright (C) 2013-2015 all contributors
-# License: AGPLv3 or later (https://www.gnu.org/licenses/agpl-3.0.txt)
+# Copyright (C) 2013-2019 all contributors
+# License: AGPL-3.0+
#
# For the -mda script (mail delivery agent)
package PublicInbox::MDA;
@@ -32,18 +32,17 @@ sub __drop_plus {
# do not allow Bcc, only Cc and To if recipient is set
sub precheck {
- my ($klass, $filter, $address) = @_;
- my Email::Simple $simple = $filter->simple;
+ my ($klass, $simple, $address) = @_;
my @mid = $simple->header('Message-ID');
return 0 if scalar(@mid) != 1;
my $mid = $mid[0];
return 0 if (length($mid) > MAX_MID_SIZE);
return 0 unless usable_str(length(''), $mid) && $mid =~ /\@/;
- return 0 unless usable_str(length('u@h'), $filter->from);
+ return 0 unless usable_str(length('u@h'), $simple->header("From"));
return 0 unless usable_str(length(':o'), $simple->header("Subject"));
return 0 unless usable_date($simple->header("Date"));
return 0 if length($simple->as_string) > MAX_SIZE;
- alias_specified($filter, $address);
+ alias_specified($simple, $address);
}
sub usable_str {
@@ -57,15 +56,15 @@ sub usable_date {
}
sub alias_specified {
- my ($filter, $address) = @_;
+ my ($simple, $address) = @_;
my @address = ref($address) eq 'ARRAY' ? @$address : ($address);
my %ok = map {
lc(__drop_plus($_)) => 1;
} @address;
- foreach my $line ($filter->cc, $filter->to) {
- my @addrs = ($line =~ /([^<\s]+\@[^>\s]+)/g);
+ foreach my $line ($simple->header('Cc'), $simple->header('To')) {
+ my @addrs = ($line =~ /([^,<\s]+\@[^,>\s]+)/g);
foreach my $addr (@addrs) {
if ($ok{lc(__drop_plus($addr))}) {
return 1;
@@ -79,10 +78,9 @@ sub set_list_headers {
my ($class, $simple, $dst) = @_;
unless (defined $simple->header('List-Id')) {
my $pa = $dst->{-primary_address};
- $simple->header_set("List-Id", "<$pa>"); # RFC2919
+ $pa =~ tr/@/./; # RFC2919
+ $simple->header_set("List-Id", "<$pa>");
}
-
- $simple->header_set($_) foreach @BAD_HEADERS;
}
1;