summaryrefslogtreecommitdiffstats
path: root/roles
diff options
context:
space:
mode:
authorGuilhem Moulin <guilhem@fripost.org>2014-01-14 05:58:33 +0100
committerGuilhem Moulin <guilhem@fripost.org>2015-06-07 02:51:33 +0200
commit0853c2afdc2ddba11692ef17bb859104d47071e0 (patch)
tree38060243978193fe71090deee666a70ae9ac21db /roles
parent677c9e95b331290fe09aa78ddd8fd6896dfce94d (diff)
Fix catchall resolution.
It has to be performed last, to give a chance to be accepted as a regular mailbox. We introduce a new, dedicated, smtpd daemon whose only purpose is to resolve catch-alls.
Diffstat (limited to 'roles')
l---------roles/IMAP/files/etc/postfix/virtual/mailbox_domains.cf2
-rw-r--r--[l---------]roles/IMAP/files/etc/postfix/virtual/mailbox_maps.cf9
-rw-r--r--roles/MX/files/etc/postfix/virtual/reserved_maps.pcre5
-rw-r--r--roles/MX/files/etc/postfix/virtual/reserved_transport_maps2
-rwxr-xr-xroles/MX/files/usr/local/sbin/reserved-alias.pl28
-rw-r--r--roles/MX/tasks/main.yml23
-rw-r--r--roles/MX/templates/etc/postfix/main.cf.j227
-rw-r--r--roles/MX/templates/etc/postfix/virtual/alias_maps.cf.j2 (renamed from roles/MX/files/etc/postfix/virtual/alias_maps.cf)0
-rw-r--r--roles/MX/templates/etc/postfix/virtual/catchall_maps.cf.j2 (renamed from roles/MX/files/etc/postfix/virtual/alias_catchall_maps.cf)0
-rw-r--r--roles/MX/templates/etc/postfix/virtual/mailbox_domains.cf.j2 (renamed from roles/MX/files/etc/postfix/virtual/mailbox_domains.cf)0
-rw-r--r--roles/MX/templates/etc/postfix/virtual/reserved_alias_maps.j24
-rw-r--r--roles/MX/templates/etc/postfix/virtual/transport_catchall_maps.cf.j28
-rw-r--r--roles/MX/templates/etc/postfix/virtual/transport_lists_maps.cf.j2 (renamed from roles/MX/files/etc/postfix/virtual/transport_lists_maps.cf)9
-rw-r--r--roles/MX/templates/etc/postfix/virtual/transport_mailbox_maps.cf.j2 (renamed from roles/MX/files/etc/postfix/virtual/mailbox_maps.cf)6
-rw-r--r--roles/MX/templates/etc/postfix/virtual/transport_reserved_maps.pcre.j26
-rw-r--r--roles/common/files/etc/postfix/master.cf10
16 files changed, 88 insertions, 51 deletions
diff --git a/roles/IMAP/files/etc/postfix/virtual/mailbox_domains.cf b/roles/IMAP/files/etc/postfix/virtual/mailbox_domains.cf
index 54363e6..05f7ed9 120000
--- a/roles/IMAP/files/etc/postfix/virtual/mailbox_domains.cf
+++ b/roles/IMAP/files/etc/postfix/virtual/mailbox_domains.cf
@@ -1 +1 @@
-../../../../../MX/files/etc/postfix/virtual/mailbox_domains.cf \ No newline at end of file
+../../../../../MX/templates/etc/postfix/virtual/mailbox_domains.cf.j2 \ No newline at end of file
diff --git a/roles/IMAP/files/etc/postfix/virtual/mailbox_maps.cf b/roles/IMAP/files/etc/postfix/virtual/mailbox_maps.cf
index 3a4d118..da1b2cf 120000..100644
--- a/roles/IMAP/files/etc/postfix/virtual/mailbox_maps.cf
+++ b/roles/IMAP/files/etc/postfix/virtual/mailbox_maps.cf
@@ -1 +1,8 @@
-../../../../../MX/files/etc/postfix/virtual/mailbox_maps.cf \ No newline at end of file
+server_host = ldapi://%2Fprivate%2Fldapi/
+version = 3
+search_base = fvl=%u,fvd=%d,ou=virtual,o=mailHosting,dc=fripost,dc=org
+scope = base
+bind = none
+query_filter = (&(objectClass=FripostVirtualUser)(fvl=%u))
+result_attribute = fvl
+result_format = OK
diff --git a/roles/MX/files/etc/postfix/virtual/reserved_maps.pcre b/roles/MX/files/etc/postfix/virtual/reserved_maps.pcre
deleted file mode 100644
index 58572d1..0000000
--- a/roles/MX/files/etc/postfix/virtual/reserved_maps.pcre
+++ /dev/null
@@ -1,5 +0,0 @@
-# These reserved aliases will always be redirected to us and the domain
-# owner.
-# TODO: check 'postmaster+test@fripost.org'
-/^(?:postmaster|abuse)(?:\+.*)?@fripost\.org$/ admin@fripost.org
-/^((?:postmaster|abuse)(?:\+.*)?)@/ $1
diff --git a/roles/MX/files/etc/postfix/virtual/reserved_transport_maps b/roles/MX/files/etc/postfix/virtual/reserved_transport_maps
deleted file mode 100644
index dce8710..0000000
--- a/roles/MX/files/etc/postfix/virtual/reserved_transport_maps
+++ /dev/null
@@ -1,2 +0,0 @@
-abuse reserved-alias:
-postmaster reserved-alias:
diff --git a/roles/MX/files/usr/local/sbin/reserved-alias.pl b/roles/MX/files/usr/local/sbin/reserved-alias.pl
index c122c6d..2c86020 100755
--- a/roles/MX/files/usr/local/sbin/reserved-alias.pl
+++ b/roles/MX/files/usr/local/sbin/reserved-alias.pl
@@ -20,10 +20,11 @@ use strict;
use Net::LDAPI;
use Net::LDAP::Util qw/escape_filter_value ldap_explode_dn escape_dn_value/;
use Authen::SASL;
+use Net::SMTP;
if (!@ARGV or grep { $_ eq '-h' or $_ eq '--help' } @ARGV) {
# Help
- print STDERR "Usage: $0 [original recipient] [additional recipient ...]\n";
+ print STDERR "Usage: $0 {original sender} {original recipient} [additional recipient ...]\n";
print STDERR "\n";
print STDERR "The message read from the standard input is redirected to 'additional recipient',\n";
print STDERR "and also forwarded to the domain owner if any. If the 'additional recipient' begins\n";
@@ -35,10 +36,13 @@ if (!@ARGV or grep { $_ eq '-h' or $_ eq '--help' } @ARGV) {
exit;
}
+# The original sender
+my $sender = shift;
+
# The original recipient
my $orig = shift;
$orig =~ /^([^@]+)\@(.+)$/
- or warn "Non fully qualified: $orig";
+ or warn "Warning: Non fully qualified: $orig";
my ($local,$domain) = ($1,$2);
# The new recipient (typically, the admin site)
@@ -60,13 +64,11 @@ my @recipients = grep { $_ and $orig ne $_ }
die "Error: Aborted delivery to '$orig' in attempt to break an alias expansion loop.\n"
unless @recipients;
-my @sendmail = ('/usr/sbin/sendmail', '-i', '-bm');
-
if (defined $domain) {
- # Look for the domain owner/postmaster
+ # Look for the domain owner or postmaster
my $ldap = Net::LDAPI->new();
$ldap->bind( sasl => Authen::SASL->new(mechanism => 'EXTERNAL') )
- or die "Couldn't bind";
+ or die "Error: Couldn't bind";
my @attrs = ( 'fripostPostmaster', 'fripostOwner' );
my $mesg = $ldap->search( base => 'fvd='.escape_dn_value($domain).','
@@ -79,16 +81,16 @@ if (defined $domain) {
, attrs => \@attrs
);
if ($mesg->code) {
- warn $mesg->error;
+ warn "Warning: ".$mesg->error;
}
elsif ($mesg->count != 1) {
# Note: this may happen for "$mydestination", but these mails
# are unlikely. We'll get a harmless warning at worst.
- warn "Something weird happened when looking up domain '".$domain.
+ warn "Warning: Something weird happened when looking up domain '".$domain.
"'. Check your ACL.";
}
else {
- my $entry = $mesg->pop_entry() // die "Cannot pop entry.";
+ my $entry = $mesg->pop_entry() // die "Error: Cannot pop entry.";
foreach (@attrs) {
my $v = $entry->get_value($_, asref => 1) or next;
foreach my $dn (@$v) {
@@ -99,7 +101,7 @@ if (defined $domain) {
push @recipients, $l.'@'.$d;
}
else {
- warn "Invalid DN: $dn"
+ warn "Warning: Invalid DN: $dn"
}
}
}
@@ -107,4 +109,8 @@ if (defined $domain) {
$ldap->unbind;
}
-exec (@sendmail, @recipients);
+my $smtp = Net::SMTP->new( 'localhost:25', Timeout => 1200 );
+$smtp->mail($sender);
+$smtp->to(@recipients, { Notify => ['FAILURE','DELAY'], SkipBad => 1 });
+$smtp->data(<STDIN>);
+$smtp->quit;
diff --git a/roles/MX/tasks/main.yml b/roles/MX/tasks/main.yml
index de6924b..2ffe08d 100644
--- a/roles/MX/tasks/main.yml
+++ b/roles/MX/tasks/main.yml
@@ -40,23 +40,24 @@
owner=root group=root
mode=0755
-- name: Copy lookups tables
- copy: src=etc/postfix/virtual/{{ item }}
- dest=/etc/postfix-{{ postfix_instance[inst].name }}/virtual/{{ item }}
- owner=root group=root
- mode=0644
+- name: Copy lookup tables
+ template: src=etc/postfix/virtual/{{ item }}.j2
+ dest=/etc/postfix-{{ postfix_instance[inst].name }}/virtual/{{ item }}
+ owner=root group=root
+ mode=0644
with_items:
- mailbox_domains.cf
- - reserved_maps.pcre
+ - reserved_alias_maps
- alias_maps.cf
- - alias_catchall_maps.cf
- - mailbox_maps.cf
+ - catchall_maps.cf
+ - transport_reserved_maps.pcre
+ - transport_mailbox_maps.cf
- transport_lists_maps.cf
- - reserved_transport_maps
+ - transport_catchall_maps.cf
- name: Compile the Reserved Transport Maps
- postmap: cmd=postalias instance={{ postfix_instance[inst].name }}
- src=/etc/postfix-{{ postfix_instance[inst].name }}/virtual/reserved_transport_maps db=cdb
+ postmap: instance={{ postfix_instance[inst].name }}
+ src=/etc/postfix-{{ postfix_instance[inst].name }}/virtual/reserved_alias_maps db=cdb
owner=root group=root
mode=0644
diff --git a/roles/MX/templates/etc/postfix/main.cf.j2 b/roles/MX/templates/etc/postfix/main.cf.j2
index d301aaf..c0da5e7 100644
--- a/roles/MX/templates/etc/postfix/main.cf.j2
+++ b/roles/MX/templates/etc/postfix/main.cf.j2
@@ -19,7 +19,7 @@ append_dot_mydomain = no
# Turn off all TCP/IP listener ports except that necessary for the mail
# exchange.
-master_service_disable = !smtp.inet inet
+master_service_disable = !smtp.inet !127.0.0.1:2599.inet inet
queue_directory = /var/spool/postfix-{{ postfix_instance[inst].name }}
data_directory = /var/lib/postfix-{{ postfix_instance[inst].name }}
@@ -52,24 +52,23 @@ relay_domains =
# Virtual transport
{% if 'LDA' in group_names %}
-virtual_transport = smtp:[127.0.0.1]:{{ LDA.port }}
+virtual_transport = smtpl:[127.0.0.1]:{{ LDA.port }}
{% else %}
-virtual_transport = smtp:[{{ LDA.IPv4 }}]:{{ LDA.port }}
+virtual_transport = smtps:[{{ LDA.host }}]:{{ LDA.port }}
{% endif %}
-transport_maps = ldap:$config_directory/virtual/transport_lists_maps.cf
+# It's a bit stupid to include part of the virtual_mailbox_maps here,
+# but we need to tell postfix to accept the recipient
+# (virtual_mailbox_maps) *before* sending away to the right machine
+# (transport_maps)
+transport_maps = pcre:$config_directory/virtual/transport_reserved_maps.pcre
+ ldap:$config_directory/virtual/transport_mailbox_maps.cf
+ ldap:$config_directory/virtual/transport_lists_maps.cf
+ ldap:$config_directory/virtual/transport_catchall_maps.cf
virtual_mailbox_domains = ldap:$config_directory/virtual/mailbox_domains.cf
-virtual_alias_maps = pcre:$config_directory/virtual/reserved_maps.pcre
+virtual_alias_maps = cdb:$config_directory/virtual/reserved_alias_maps
ldap:$config_directory/virtual/alias_maps.cf
- ldap:$config_directory/virtual/alias_catchall_maps.cf
-virtual_mailbox_maps = ldap:$config_directory/virtual/mailbox_maps.cf
- # it's a bit stupid to lookup for lists here
- # and in transport, but we need to tell
- # postfix to accept the recipient
- # (virtual_mailbox_maps) *before* sending away
- # to the list server (transport_maps)
- ldap:$config_directory/virtual/transport_lists_maps.cf
-mailbox_transport_maps = cdb:$config_directory/virtual/reserved_transport_maps
+virtual_mailbox_maps = $transport_maps
# Don't rewrite remote headers
local_header_rewrite_clients =
diff --git a/roles/MX/files/etc/postfix/virtual/alias_maps.cf b/roles/MX/templates/etc/postfix/virtual/alias_maps.cf.j2
index 8e3a778..8e3a778 100644
--- a/roles/MX/files/etc/postfix/virtual/alias_maps.cf
+++ b/roles/MX/templates/etc/postfix/virtual/alias_maps.cf.j2
diff --git a/roles/MX/files/etc/postfix/virtual/alias_catchall_maps.cf b/roles/MX/templates/etc/postfix/virtual/catchall_maps.cf.j2
index f8324f6..f8324f6 100644
--- a/roles/MX/files/etc/postfix/virtual/alias_catchall_maps.cf
+++ b/roles/MX/templates/etc/postfix/virtual/catchall_maps.cf.j2
diff --git a/roles/MX/files/etc/postfix/virtual/mailbox_domains.cf b/roles/MX/templates/etc/postfix/virtual/mailbox_domains.cf.j2
index 74304a4..74304a4 100644
--- a/roles/MX/files/etc/postfix/virtual/mailbox_domains.cf
+++ b/roles/MX/templates/etc/postfix/virtual/mailbox_domains.cf.j2
diff --git a/roles/MX/templates/etc/postfix/virtual/reserved_alias_maps.j2 b/roles/MX/templates/etc/postfix/virtual/reserved_alias_maps.j2
new file mode 100644
index 0000000..fe04715
--- /dev/null
+++ b/roles/MX/templates/etc/postfix/virtual/reserved_alias_maps.j2
@@ -0,0 +1,4 @@
+# RFC 822 section 6.3 and RFC 2142 section 4 mandatory aliases are
+# forwarded to the admin team.
+postmaster@fripost.org admin@fripost.org
+abuse@fripost.org admin@fripost.org
diff --git a/roles/MX/templates/etc/postfix/virtual/transport_catchall_maps.cf.j2 b/roles/MX/templates/etc/postfix/virtual/transport_catchall_maps.cf.j2
new file mode 100644
index 0000000..cc189cf
--- /dev/null
+++ b/roles/MX/templates/etc/postfix/virtual/transport_catchall_maps.cf.j2
@@ -0,0 +1,8 @@
+server_host = ldapi://%2Fprivate%2Fldapi/
+version = 3
+search_base = fvd=%d,ou=virtual,o=mailHosting,dc=fripost,dc=org
+scope = base
+bind = none
+query_filter = (&(objectClass=FripostVirtualDomain)(fvd=%d)(fripostOptionalMaildrop=*))
+result_attribute = fvd
+result_format = smtpl:[127.0.0.1]:2599
diff --git a/roles/MX/files/etc/postfix/virtual/transport_lists_maps.cf b/roles/MX/templates/etc/postfix/virtual/transport_lists_maps.cf.j2
index 27c93d1..6a0965f 100644
--- a/roles/MX/files/etc/postfix/virtual/transport_lists_maps.cf
+++ b/roles/MX/templates/etc/postfix/virtual/transport_lists_maps.cf.j2
@@ -5,7 +5,8 @@ scope = base
bind = none
query_filter = (&(|(objectClass=FripostVirtualList)(objectClass=FripostVirtualListCommand))(fvl=%u))
result_attribute = fvl
-# We use these maps for both virtual mailboxes and transport (in the
-# former case the result is ignored, only the existence of a match
-# matters.)
-result_format = smtp:[127.0.0.1]:2345
+{% if 'lists' in group_names %}
+result_format = smtpl:[127.0.0.1]:{{ lists.port }}
+{% else %}
+result_format = smtps:[{{ lists.host }}]:{{ lists.port }}
+{% endif %}
diff --git a/roles/MX/files/etc/postfix/virtual/mailbox_maps.cf b/roles/MX/templates/etc/postfix/virtual/transport_mailbox_maps.cf.j2
index da1b2cf..3e003db 100644
--- a/roles/MX/files/etc/postfix/virtual/mailbox_maps.cf
+++ b/roles/MX/templates/etc/postfix/virtual/transport_mailbox_maps.cf.j2
@@ -5,4 +5,8 @@ scope = base
bind = none
query_filter = (&(objectClass=FripostVirtualUser)(fvl=%u))
result_attribute = fvl
-result_format = OK
+{% if 'LDA' in group_names %}
+result_format = smtpl:[127.0.0.1]:{{ LDA.port }}
+{% else %}
+result_format = smtps:[{{ LDA.host }}]:{{ LDA.port }}
+{% endif %}
diff --git a/roles/MX/templates/etc/postfix/virtual/transport_reserved_maps.pcre.j2 b/roles/MX/templates/etc/postfix/virtual/transport_reserved_maps.pcre.j2
new file mode 100644
index 0000000..e240e91
--- /dev/null
+++ b/roles/MX/templates/etc/postfix/virtual/transport_reserved_maps.pcre.j2
@@ -0,0 +1,6 @@
+if !/@fripost\.org$/
+# For other domains, RFC 822 section 6.3 and RFC 2142 section 4
+# mandatory aliases are forwarded to OUR admin team and to the domain
+# owner or postmaster, if there are any.
+/^(?:postmaster|abuse)(?:\+.*)?@/ reserved-alias:
+endif
diff --git a/roles/common/files/etc/postfix/master.cf b/roles/common/files/etc/postfix/master.cf
index 38b2ecb..17613b0 100644
--- a/roles/common/files/etc/postfix/master.cf
+++ b/roles/common/files/etc/postfix/master.cf
@@ -26,6 +26,10 @@ flush unix n - - 1000? 0 flush
proxymap unix - - n - - proxymap
proxywrite unix - - n - 1 proxymap
smtp unix - - - - - smtp
+smtpl unix - - - - - smtp
+ -o smtp_bind_address=127.0.0.1
+smtps unix - - - - - smtp
+ -o smtp_bind_address=172.16.0.1
relay unix - - - - - smtp
# -o smtp_helo_timeout=5 -o smtp_connect_timeout=5
showq unix n - - - - showq
@@ -37,8 +41,12 @@ virtual unix - n n - - virtual
lmtp unix - - - - - lmtp
anvil unix - - - - 1 anvil
scache unix - - - - 1 scache
+127.0.0.1:2599 inet n - - - - smtpd
+ -o cleanup_service_name=cleanup-catchall
+cleanup-catchall unix n - - - 0 cleanup
+ -o virtual_alias_maps=cdb:$config_directory/virtual/reserved_alias_maps,ldap:$config_directory/virtual/alias_maps.cf,ldap:/etc/postfix-mx/virtual/catchall_maps.cf
reserved-alias unix - n n - - pipe
- flags=Rhu user=mail argv=/usr/local/sbin/reserved-alias.pl ${original_recipient} @fripost.org
+ flags=Rhu user=nobody argv=/usr/local/sbin/reserved-alias.pl ${sender} ${original_recipient} @fripost.org
amavisfeed unix - - n - 2 lmtp
-o lmtp_destination_recipient_limit=1000
-o lmtp_send_xforward_command=yes