and that didn't work
[mirror/dsa-puppet.git] / modules / exim / templates / eximconf.erb
index 6dd0a37..a0c95c0 100644 (file)
 #           flushing' operations, but should be populated with a list
 #           of trusted machines. Wildcards are not permitted
 #  bsmtp_domains - Domains that we deliver locally via bsmtp
-<%=
-out = ""
-if nodeinfo['mailrelay']
-  out = '
+<% if nodeinfo['mailrelay'] %>
 #  mailhubdomains - Domains for which we are the MX, but the mail is relayed
 #           elsewhere.  This is designed for use with small volume or
 #           restricted machines that need to use a smarthost for mail
 #           traffic.  We will relay for them based on ssl cert validation
 #           but we need to teach exim how to route the mail to them.  This is
 #           that list.
-'
-end
-out
-%>
+<% end %>
 # Exim's wildcard mechanism is a bit odd in that to say "any address in
 # debian.org including debian.org" you must use two patterns,
 #   *.debian.org
@@ -81,15 +75,9 @@ out
 #                    MAIN CONFIGURATION SETTINGS                     #
 ######################################################################
 
-<%=
-out=''
-if nodeinfo.has_key?('heavy_exim') and not nodeinfo['heavy_exim'].empty?
-  out = "
+<% if nodeinfo.has_key?('heavy_exim') and not nodeinfo['heavy_exim'].empty? %>
 perl_startup = do '/etc/exim4/exim_surbl.pl'
-"
-end
-out
-%>
+<% end %>
 
 # These options specify the Access Control Lists (ACLs) that
 # are used for incoming SMTP messages - after the RCPT and DATA
@@ -98,13 +86,9 @@ out
 acl_smtp_helo = check_helo
 acl_smtp_rcpt = ${if ={$interface_port}{587} {check_submission}{check_recipient}}
 acl_smtp_data = check_message
-<%=
-out=''
-if nodeinfo.has_key?('heavy_exim') and not nodeinfo['heavy_exim'].empty?
-  out = "acl_smtp_mime = acl_check_mime"
-end
-out
-%>
+<% if nodeinfo.has_key?('heavy_exim') and not nodeinfo['heavy_exim'].empty? %>
+acl_smtp_mime = acl_check_mime
+<% end %>
 acl_smtp_predata = acl_check_predata
 
 # accept domain literal syntax in e-mail addresses. To actually make use of
@@ -134,38 +118,30 @@ localpartlist local_only_users = lsearch;/etc/exim4/localusers
 
 localpartlist postmasterish = postmaster : abuse : hostmaster : root
 
+hostlist debianhosts = 127.0.0.1 : /var/lib/misc/thishost/debianhosts
+
+hostlist reservedaddrs = <%= nodeinfo['reservedaddrs'] %>
+
+<% if nodeinfo['mailrelay'] %>
 # Domains we relay for; that is domains that aren't considered local but we 
 # accept mail for them.
-hostlist debianhosts = 127.0.0.1 : net-lsearch;/var/lib/misc/thishost/debianhosts
-<%=
-out = ""
-if nodeinfo['mailrelay']
-  out = '
 domainlist mailhubdomains = lsearch;/etc/exim4/manualroute
-'
-end
-out
-%>
-
-hostlist reservedaddrs = <%= nodeinfo['reservedaddrs'] %>
+<% end %>
 
-<%= out = ""
-if has_variable?("exim_ssl_certs") && exim_ssl_certs == "true"
-out = "tls_certificate = /etc/exim4/ssl/thishost.crt
+<% if has_variable?("exim_ssl_certs") && exim_ssl_certs == "true" %>
+tls_certificate = /etc/exim4/ssl/thishost.crt
 tls_privatekey = /etc/exim4/ssl/thishost.key
 tls_try_verify_hosts = *
 tls_verify_certificates = /etc/exim4/ssl/ca.crt
-tls_crl = /etc/exim4/ssl/ca.crl"
-end
-out
-%>
+tls_crl = /etc/exim4/ssl/ca.crl
+<% end %>
 
 # The setting below causes Exim to do a reverse DNS lookup on all incoming
 # IP calls, in order to get the true host name. If you feel this is too
 # expensive, you can specify the networks for which a lookup is done, or
 # remove the setting entirely.
 host_lookup = *
-dns_ipv4_lookup = !localhost
+# dns_ipv4_lookup = !localhost (disabled upon sgrans request, zobel, 2010-03-16)
 
 # If this option is set, then any process that is running as one of the
 # listed users may pass a message to Exim and specify the sender's
@@ -223,12 +199,9 @@ queue_only_load = 5
 <% end %>
 queue_list_requires_admin = false
 
-<%= out  = ""
-if has_variable?("clamd") && clamd == "true"
-  out = "av_scanner = clamd:/var/run/clamav/clamd.ctl"
-end
-out
-%>
+<% if has_variable?("clamd") && clamd == "true" %>
+av_scanner = clamd:/var/run/clamav/clamd.ctl
+<% end %>
 
 <%= 
 ports = []
@@ -255,12 +228,9 @@ admin_groups = adm
 remote_sort_domains = *.debian.org:*.debian.net
 
 pipelining_advertise_hosts = !*
-<%= out = ""
-if has_variable?("exim_ssl_certs") && exim_ssl_certs == "true"
-out = 'tls_advertise_hosts = *'
-end
-out
-%>
+<% if has_variable?("exim_ssl_certs") && exim_ssl_certs == "true" %>
+tls_advertise_hosts = *
+<% end %>
 smtp_enforce_sync = true
 
 log_selector = +tls_cipher +tls_peerdn +queue_time +deliver_time +smtp_connection +smtp_incomplete_transaction +smtp_confirmation
@@ -293,6 +263,14 @@ RT_QUEUE_MAP = /srv/rt.debian.org/mail/rt_queue_map
 ######################################################################
 begin acl
 
+acl_spamlovers:
+  # There are a few profiles that don't want much smtp time checking of
+  # mail.  It's easier to track them in one place
+
+  accept  condition      = ${if eq {$acl_m_prf}{PopconMail}}
+  accept  condition      = ${if eq {$acl_m_prf}{BugsMail}}
+  deny
+
 acl_getprofile:
   # This is a bad hack to reset the variable, by defining it be something
   # never referenced.
@@ -311,34 +289,28 @@ acl_getprofile:
 
   accept  condition      = ${if eq {$acl_m_rprf}{}{no}{yes}}
 
-<%=
-out=''
-if nodeinfo['rtmaster']
-  out='
+<% if nodeinfo['rtmaster'] %>
   warn    domains        = rt.debian.org
           set acl_m_rprf = RTMail
 
   accept  condition      = ${if eq {$acl_m_rprf}{}{no}{yes}}
-'
-end
-out
-%>
-<%=
-out = ''
-if nodeinfo['packagesmaster']
-  out = '
+
+<% end %>
+<% if nodeinfo['bugsmx'] %>
+  warn    domains        = bugs.debian.org
+          set acl_m_rprf = BugsMail
+
+  accept  condition      = ${if eq {$acl_m_rprf}{}{no}{yes}}
+
+<% end %>
+<% if nodeinfo['packagesmaster'] %>
   warn    domains        = packages.debian.org
           set acl_m_rprf = PackagesMail
 
   accept  condition      = ${if eq {$acl_m_rprf}{}{no}{yes}}
-'
-end
-out
-%>
-<%=
-out = ''
-if nodeinfo['packagesqamaster']
-  out='
+
+<% end %>
+<% if nodeinfo['packagesqamaster'] %>
   warn    recipients     = owner@packages.qa.debian.org : postmaster@packages.qa.debian.org
           set acl_m_rprf = PTSOwner
 
@@ -355,10 +327,8 @@ if nodeinfo['packagesqamaster']
           set acl_m_rprf = PTSMail
 
   accept  condition      = ${if eq {$acl_m_rprf}{}{no}{yes}}
-'
-end
-out
-%>
+
+<% end %>
   warn    recipients     = change@db.debian.org : changes@db.debian.org : chpasswd@db.debian.org : ping@db.debian.org : recommend@nm.debian.org
           set acl_m_rprf = DBSignedMail
 
@@ -400,19 +370,13 @@ check_helo:
 
   warn    set acl_c_scr    = 0
 
-<%= 
-out = ""
-if nodeinfo['mailrelay']
-  out = "  accept  verify   = certificate"
-end
-out
-%>
+<% if nodeinfo['mailrelay'] %>
+  accept  verify   = certificate
 
-<%= 
-if nodeinfo['smarthost'].empty?
-  out = '
+<% end %>
+<% if nodeinfo['smarthost'].empty? %>
   # These are in HELO acl so that they are only run once.  They increment a counter,
-  # so we don\'t want it to increment per rcpt to.
+  # so we don't want it to increment per rcpt to.
 
   warn    dnslists       = list.dnswl.org&0.0.0.3
           log_message    = Hit on list.dnswl.org for $sender_host_address
@@ -447,7 +411,7 @@ if nodeinfo['smarthost'].empty?
           dnslists       = dul.dnsbl.sorbs.net
           set acl_c_scr  = ${eval:$acl_c_scr+15}
 
-  # If the sender\'s helo name is empty, the message will be rejected later
+  # If the sender's helo name is empty, the message will be rejected later
   # because the helo is empty.  If the rDNS lookup failed, we are already
   # going to greylist them, so no sense worrying about it here.  Finally,
   # if rDNS does not match helo name (both lower cased first), greylist.
@@ -456,7 +420,7 @@ if nodeinfo['smarthost'].empty?
           condition      = ${if eq {$host_lookup_failed}{1}{no}{yes}}
           condition      = ${if def:sender_helo_name {yes}{no}}
           condition      = ${if eq {${lc:$sender_helo_name}}{${lc:$sender_host_name}}{no}{yes}}
-          log_message    = HELO doesn\'t match rDNS
+          log_message    = HELO doesn't match rDNS
           set acl_c_scr  = ${eval:$acl_c_scr+8}
 
   # Regexes of doom
@@ -477,25 +441,21 @@ if nodeinfo['smarthost'].empty?
        set acl_c_scr   = ${eval:$acl_c_scr+7}
 
   # Random HELO (run of 7 consonants) (constructed by viruses).  We purposefully
-  # skip matching on machines named .*smtp.*, since that\'s 4 already.  This is a fairly
-  # naive test, so it\'s not worth much
+  # skip matching on machines named .*smtp.*, since that's 4 already.  This is a fairly
+  # naive test, so it's not worth much
 
   warn condition       = ${if match {${lc:$sender_helo_name}}{smtp}{no}{yes}}
        condition       = ${if match {${lc:$sender_helo_name}}{\N^[a-z0-9]+\.[a-z]+$\N}}
        condition       = ${if match {${lc:$sender_helo_name}}{\N.*[bcdfghjklmnpqrstvwxz]{7,}.*\.[a-z]+$\N}}
        log_message     = random HELO
        set acl_c_scr   = ${eval:$acl_c_scr+5}
-'
-else
-  out = '
+
+<% else %>
   drop !hosts          = +debianhosts
        log_message     = mail from non-d.o host
        message         = Interesting.  I doubt that should have happened.
-'
-end
-out
-%>
 
+<% end %>
   # Implicit, but simpler to just say it
   accept
 
@@ -506,14 +466,10 @@ check_submission:
   # We do this by testing for an empty sending host field.
   accept  hosts = : 127.0.0.1
 
-<%= 
-out = ""
-if nodeinfo['mailrelay']
-  out = "  accept  verify   = certificate"
-end
-out
-%>
+<% if nodeinfo['mailrelay'] %>
+  accept  verify   = certificate
 
+<% end %>
   # Defer after too many bad RCPT TO's.  Legit MTAs will retry later.
   # This is a rough pass at preventing addres harvesting or other mail blasts.
 
@@ -531,18 +487,12 @@ out
           endpass
          verify   = recipient
 
-<%=
-out = ""
-if nodeinfo['mailrelay']
-  out = '
+<% if nodeinfo['mailrelay'] %>
   accept  domains  = +mailhubdomains
           endpass
          verify   = recipient/callout=30s,defer_ok,use_sender,no_cache
-'
-end
-out
-%>
 
+<% end %>
   accept  domains  = +submission_domains
           endpass
          verify   = recipient
@@ -552,14 +502,10 @@ out
 #!!# ACL that is used after the RCPT command
 check_recipient:
 
-<%=
-out = ""
-if nodeinfo['mailrelay']
-  out = "  accept  verify   = certificate"
-end
-out
-%>
+<% if nodeinfo['mailrelay'] %>
+  accept  verify   = certificate
 
+<% end %>
   warn    acl           = acl_getprofile
           condition     = ${if eq{$acl_m_prf}{}}
           set acl_m_prf = $acl_m_rprf
@@ -571,25 +517,25 @@ out
   # This is a rough pass at preventing addres harvesting or other mail blasts.
 
   defer  log_message   = Too many bad recipients ${eval:$rcpt_fail_count} out of $rcpt_count
-         condition     = ${if eq {$acl_m_prf}{PopconMail}{no}{yes}}
+         !acl          = acl_spamlovers
          message       = Too many bad recipients, try again later
          !hosts        = +debianhosts
          condition     = ${if > {${eval:$rcpt_fail_count}}{3}{yes}{no}}
 
   # Dump spambots that are so stupid they say helo as our IP address
 
-  drop !hosts          = +debianhosts
-        condition      = ${if eq {$acl_m_prf}{PopconMail}{no}{yes}}
-       condition       = ${if eq {$sender_helo_name}{$interface_address}{yes}{no}}
-       message         = HELO mismatch Forged HELO for ($sender_helo_name)
+  drop   !hosts        = +debianhosts
+         !acl          = acl_spamlovers
+         condition     = ${if eq {$sender_helo_name}{$interface_address}{yes}{no}}
+         message       = HELO mismatch Forged HELO for ($sender_helo_name)
 
   # Also for spambots that say helo as us or one of our domains
 
-  drop !hosts          = +debianhosts
-        condition      = ${if eq {$acl_m_prf}{PopconMail}{no}{yes}}
-       condition       = ${if match_domain{$sender_helo_name}{$primary_hostname:+handled_domains}}
-       condition       = ${if !match{$sender_host_name}{${rxquote:$sender_helo_name}\N$\N}}
-       message         = HELO mismatch Forged HELO for ($sender_helo_name)
+  drop   !hosts        = +debianhosts
+         !acl          = acl_spamlovers
+         condition     = ${if match_domain{$sender_helo_name}{$primary_hostname:+handled_domains}}
+         condition     = ${if !match{$sender_host_name}{${rxquote:$sender_helo_name}\N$\N}}
+         message       = HELO mismatch Forged HELO for ($sender_helo_name)
 
   # This logic gives you a list of commonly forged domains in helo to reject against
 
@@ -600,31 +546,31 @@ out
   # This is a failsafe in case DNS fails - we defer instead of hard reject if they 
   # say helo as a name in the list but we can't look them up
 
-  defer !hosts         = +debianhosts
-        condition      = ${if eq {$acl_m_prf}{PopconMail}{no}{yes}}
-        condition      = ${if eq{$acl_m_frg}{}{no}{yes}}
-        condition      = ${if eq{$sender_host_name}{}{yes}{no}}
-        condition      = ${if eq{$host_lookup_failed}{1}{no}{yes}}
-        message        = Access temporarily denied. Resolve failed PTR for $sender_host_address
+  defer  !hosts        = +debianhosts
+         !acl          = acl_spamlovers
+         condition     = ${if eq{$acl_m_frg}{}{no}{yes}}
+         condition     = ${if eq{$sender_host_name}{}{yes}{no}}
+         condition     = ${if eq{$host_lookup_failed}{1}{no}{yes}}
+         message       = Access temporarily denied. Resolve failed PTR for $sender_host_address
 
   # If DNS works, go ahead and reject them
 
-  drop  !hosts         = +debianhosts
-        condition      = ${if eq {$acl_m_prf}{PopconMail}{no}{yes}}
-        condition      = ${if and { {!eq{$acl_m_frg}{}}{!match{$sender_host_name}{${rxquote:$acl_m_frg}\N$\N}}}{yes}{no}}
-        message        = HELO mismatch Forged HELO for ($sender_helo_name)
+  drop   !hosts        = +debianhosts
+         !acl          = acl_spamlovers
+         condition     = ${if and { {!eq{$acl_m_frg}{}}{!match{$sender_host_name}{${rxquote:$acl_m_frg}\N$\N}}}{yes}{no}}
+         message       = HELO mismatch Forged HELO for ($sender_helo_name)
 
   # disabled accounts don't even get local mail.
-  deny    local_parts   = lsearch;/var/lib/misc/$primary_hostname/mail-disable
-          domains       = +local_domains
-         message       = ${lookup{$local_part}lsearch{/var/lib/misc/$primary_hostname/mail-disable}{$value}}
-
-  deny    domains       = +virtual_domains
-          local_parts   = ${if exists {${extract{directory}{VDOMAINDATA}{${value}/localonly}}}\
-                                      {${extract{directory}{VDOMAINDATA}{${value}/localonly}}}\
-                                      {}}
-          hosts         = !+debianhosts
-          message       = mail for <$local_part@$domain> only accepted from debian.org machines
+  deny   local_parts   = lsearch;/var/lib/misc/$primary_hostname/mail-disable
+         domains       = +local_domains
+        message       = ${lookup{$local_part}lsearch{/var/lib/misc/$primary_hostname/mail-disable}{$value}}
+
+  deny   domains       = +virtual_domains
+         local_parts   = ${if exists {${extract{directory}{VDOMAINDATA}{${value}/localonly}}}\
+                                     {${extract{directory}{VDOMAINDATA}{${value}/localonly}}}\
+                                     {}}
+         hosts         = !+debianhosts
+         message       = mail for <$local_part@$domain> only accepted from debian.org machines
   # Accept if the source is local SMTP (i.e. not over TCP/IP).
   # We do this by testing for an empty sending host field.
   accept  hosts = :
@@ -654,28 +600,18 @@ out
   warn    condition     = ${if eq{$acl_m_prf}{localonly}}
           set acl_m_lrc = ${if eq{$acl_m_lrc}{}{$local_part@$domain}{$acl_m_lrc, $local_part@$domain}}
 
-<%=
-out=''
-if 0 == 1:
-out='
+<% if 0 == 1 %>
   deny    message  = address $sender_host_address is listed in $dnslist_domain; $dnslist_text
          hosts    = !+debianhosts
           dnslists = rbl.debian.net : rbl.debian.net/$sender_address_domain
-'
-end
-out
-%>
-<%=
-out=''
-if nodeinfo['packagesmaster']
-  out='
+
+<% end %>
+<% if nodeinfo['packagesmaster'] %>
   warn    condition      = ${if eq {$acl_m_prf}{PackagesMail}}
           condition      = ${if eq {$sender_address}{$local_part@$domain}}
           message        = X-Packages-FromTo-Same: yes
-'
-end
-out
-%>
+
+<% end %>
   deny    condition      = ${if eq {$acl_m_prf}{PopconMail}{no}{yes}}
           !verify        = sender
 
@@ -684,17 +620,15 @@ out
           condition      = ${if >{${eval:$acl_c_scr+0}}{0}}
           ratelimit      = 10 / 60m / per_rcpt / $sender_host_address
           message        = slow down (no reverse dns, mismatched ehlo, dialup, or in blacklists)
-<%=
-out = ""
-if has_variable?("policydweight") && policydweight == "true"
-out = '
-  # Check with policyd-weight - this only works with a version after etch\'s,
-  # sadly.  etch\'s version attempts to hold the socket open, since that\'s what
+
+<% if has_variable?("policydweight") && policydweight == "true" %>
+  # Check with policyd-weight - this only works with a version after etch's,
+  # sadly.  etch's version attempts to hold the socket open, since that's what
   # postfix expects.  Exim, on the other hand, expects the remote side to close
-  # the socket when it\'s finished sending data, so it see each transaction as
-  # an incomplete read.  I\'m sure there\'s a way we could force exim to do
+  # the socket when it's finished sending data, so it see each transaction as
+  # an incomplete read.  I'm sure there's a way we could force exim to do
   # something sick and clever to force either the interpretation or the socket
-  # closure, but I\'m fairly sure it\'s now worth it, since the backport of
+  # closure, but I'm fairly sure it's now worth it, since the backport of
   # policyd-weight is trivial.
   warn  !hosts         = +debianhosts
         condition      = ${if eq {$acl_m_prf}{PopconMail}{no}{yes}}
@@ -731,7 +665,7 @@ out = '
         message        = $acl_m_mes
         condition      = ${if eq{$acl_m_act}{PREPEND}{yes}{no}}
 
-  # Write log message, if policyd-weight can\'t run checks
+  # Write log message, if policyd-weight can't run checks
   warn  !hosts         = +debianhosts
         condition      = ${if eq {$acl_m_prf}{PopconMail}{no}{yes}}
         log_message    = policyd-weight message: $acl_m_mes
@@ -748,24 +682,14 @@ out = '
          condition      = ${if eq {$acl_m_prf}{PopconMail}{no}{yes}}
          message        = policyd-weight said: $acl_m_mes
          condition      = ${if eq{$acl_m_act}{450}{yes}{no}}
-'
-end
-out
-%>
-<%=
-out=''
-if nodeinfo['rtmaster']
-  out='
+
+<% end %>
+<% if nodeinfo['rtmaster'] %>
   warn    condition     = ${if eq{$acl_m_prf}{RTMail}}
           set acl_m12   = ${if def:acl_m12 {$acl_m12} {${if or{{match{$local_part}{\N[^+]+\+\d+\N}}{match{$local_part}{\N[^+]+\+new\N}}} {RTMailRecipientHasSubaddress}}}}
-'
-end
-out
-%>
-<%=
-out = ""
-if has_variable?("greylistd") && greylistd == "true"
-  out = '
+
+<% end %>
+<% if has_variable?("greylistd") && greylistd == "true" %>
   defer
     message  = $sender_host_address is not yet authorized to deliver mail from <$sender_address> to <$local_part@$domain>.
     log_message = greylisted.
@@ -789,11 +713,10 @@ if has_variable?("greylistd") && greylistd == "true"
                                   $sender_address \
                                   $local_part@$domain}\
                                  {5s}{}{false}}
-'
-elsif has_variable?("postgrey") && postgrey == "true"
-  out = '
+
+<% elsif has_variable?("postgrey") && postgrey == "true" %>
   # next three are greylisting, inspired by http://www.bebt.de/blog/debian/archives/2006/07/30/T06_12_27/index.html
-  # this adds acl_m_grey if there isn\'t one (so unique per message)
+  # this adds acl_m_grey if there isn't one (so unique per message)
   warn
     !senders       = :
     !hosts         = : +debianhosts : WHITELIST
@@ -822,7 +745,7 @@ elsif has_variable?("postgrey") && postgrey == "true"
                          ${readsocket{/var/run/postgrey/socket}{$acl_m_pgr}\
                                {5s}{}{action=DUNNO}}\
                      }{action=}{}}
-    message        = ${sg{$acl_m_pgr}{^\\\\w+\\\\s*}{}}
+    message        = ${sg{$acl_m_pgr}{^\\w+\\s*}{}}
     log_message    = greylisted.
     condition      = ${if eq{${uc:${substr{0}{5}{$acl_m_pgr}}}}{DEFER}}
 
@@ -835,11 +758,9 @@ elsif has_variable?("postgrey") && postgrey == "true"
     domains        = +handled_domains
     local_parts    = GREYLIST_LOCAL_PARTS
     condition      = ${if eq{${uc:${substr_0_7:$acl_m_pgr}}}{PREPEND}}
-    message        = ${sg{$acl_m_pgr}{^\\\\w+\\\\s*}{}}
-'
-end
-out
-%>
+    message        = ${sg{$acl_m_pgr}{^\\w+\\s*}{}}
+
+<% end %>
 
   accept  local_parts   = +postmasterish
           domains       = +handled_domains
@@ -852,10 +773,7 @@ out
          senders       = ${if exists{/etc/exim4/blacklist}{/etc/exim4/blacklist}{}}
          message       = We have blacklisted <$sender_address>.  Please stop mailing us
 
-<%= 
-out = ""
-if nodeinfo['smarthost'].empty?
-  out = '
+<% if nodeinfo['smarthost'].empty? %>
   deny    message  = host $sender_host_address is listed in $dnslist_domain; see $dnslist_text
           dnslists = ${if match_domain{$domain}{+virtual_domains}\
                     {${if exists {${extract{directory}{VDOMAINDATA}{${value}/rbllist}}}\
@@ -864,11 +782,8 @@ if nodeinfo['smarthost'].empty?
                     ${lookup{$local_part}lsearch{/var/lib/misc/$primary_hostname/mail-rbl}{$value}{}}}}
          domains       = +handled_domains
          !hosts        = +debianhosts : WHITELIST
-'
-end
-out
-%>
 
+<% end %>
   deny    message  = domain $sender_address_domain is listed in $dnslist_domain; see $dnslist_text
           dnslists = ${if match_domain{$domain}{+virtual_domains}\
                     {${if exists {${extract{directory}{VDOMAINDATA}{${value}/rhsbllist}}}\
@@ -878,10 +793,7 @@ out
          domains       = +handled_domains
          !hosts        = +debianhosts : WHITELIST
 
-<%= 
-out = ""
-if nodeinfo['smarthost'].empty?
-  out = '
+<% if nodeinfo['smarthost'].empty? %>
   deny    domains  = +handled_domains
           local_parts   = ${if match_domain{$domain}{+virtual_domains}\
                           {${if exists {${extract{directory}{VDOMAINDATA}{${value}/callout_users}}}\
@@ -890,22 +802,14 @@ if nodeinfo['smarthost'].empty?
                           ${lookup{$local_part}lsearch{/var/lib/misc/$primary_hostname/mail-callout}{$local_part}{}}}}
           !hosts   = +debianhosts : WHITELIST
          !verify  = sender/callout=90s,maxwait=300s
-'
-end
-out
-%>
 
-<%=
-out = ""
-if nodeinfo['mailrelay']
-  out = '
+<% end %>
+<% if nodeinfo['mailrelay'] %>
   accept  domains  = +mailhubdomains
           endpass
          verify   = recipient/callout=30s,defer_ok,use_sender,no_cache
-'
-end
-out
-%>
+
+<% end %>
   accept  domains  = +handled_domains
           endpass
          verify   = recipient/defer_ok
@@ -916,10 +820,7 @@ out
 
   deny    message = relay not permitted
 
-<%=
-out=''
-if nodeinfo.has_key?('heavy_exim') and not nodeinfo['heavy_exim'].empty?
-out='
+<% if nodeinfo.has_key?('heavy_exim') and not nodeinfo['heavy_exim'].empty? %>
 acl_check_mime:
 
  discard condition     = ${if <{$message_size}{256000}}
@@ -943,10 +844,8 @@ acl_check_mime:
          message       = X-Surbl-Hit: $primary_hostname: $acl_m_srb
 
   accept
-'
-end
-out
-%>
+
+<% end %>
 
 acl_check_predata:
   deny   condition     = ${if eq{$acl_m_prf}{localonly}}
@@ -957,31 +856,21 @@ acl_check_predata:
 
 #!!# ACL that is used after the DATA command
 check_message:
-<%=
-out=''
-if nodeinfo['rtmaster']
-  out='
+<% if nodeinfo['rtmaster'] %>
   deny    condition = ${if eq {$acl_m_prf}{RTMail}}
           condition = ${if and{{!match {${lc:$rh_Subject:}} {debian rt}} \
                                {!match {${lc:$rh_Subject:]}} {\N\[rt.debian.org \N}} \
                                {!match {$acl_m12}{RTMailRecipientHasSubaddress}}}}
           message  = messages to the Request Tracker system require a subject tag or a subaddress
-'
-end
-out
-%>
-<%=
-out=''
-if nodeinfo['packagesqamaster']
-  out='
+
+<% end %>
+<% if nodeinfo['packagesqamaster'] %>
   deny    !hosts  = +debianhosts : 217.196.43.134
           condition = ${if eq {$acl_m_prf}{PTSMail}}
           condition = ${if def:h_X-PTS-Approved:{false}{true}}
           message   = messages to the PTS require an X-PTS-Approved header
-'
-end
-out
-%>
+
+<% end %>
   deny    condition      = ${if eq {$acl_m_prf}{DBSignedMail}}
           condition      = ${if and {{!match {$message_body}{PGP MESSAGE}}              \
                                      {!match {$message_body}{PGP SIGNED MESSAGE}}       \
@@ -1014,10 +903,7 @@ out
           condition       = ${if eq {$acl_m_prf}{PopconMail}{no}{yes}}
          message         = Your mailer is not RFC 2047 compliant: message rejected
 
-<%=
-out = ""
-if has_variable?("clamd") && clamd == "true"
-out = '
+<% if has_variable?("clamd") && clamd == "true" %>
   discard condition       = ${if eq {$acl_m_prf}{blackhole}}
           demime          = *
           malware         = */defer_ok
@@ -1033,14 +919,9 @@ out = '
          demime          = *
           malware         = */defer_ok
           message         = X-malware detected: $malware_name
-'
-end
-out
-%>
-<%=
-out=''
-if nodeinfo.has_key?('heavy_exim') and not nodeinfo['heavy_exim'].empty?
-out='
+
+<% end %>
+<% if nodeinfo.has_key?('heavy_exim') and not nodeinfo['heavy_exim'].empty? %>
  discard condition     = ${if <{$message_size}{256000}}
          condition     = ${if eq {$acl_m_prf}{blackhole}}
          set acl_m_srb = ${perl{surblspamcheck}}
@@ -1061,19 +942,13 @@ out='
          condition     = ${if eq{$acl_m_srb}{false}{no}{yes}}
          message       = X-Surbl-Hit: $primary_hostname: $acl_m_srb
 
-'
-end
-out
-%>
+<% end %>
   # Check header_sender except for survey@popcon.d.o
   deny    condition    = ${if eq{$acl_m_prf}{PopconMail}{false}{true}}
           !verify      = header_sender
           message      = No valid sender found in the From:, Sender: and Reply-to: headers
 
-<%=
-out = ""
-if nodeinfo['packagesmaster']
-  out = '
+<% if nodeinfo['packagesmaster'] %>
   deny  message        = Congratulations, you scored $spam_score points.
         log_message    = spam: $spam_score points.
         condition      = ${if eq {$acl_m_prf}{PackagesMail}}
@@ -1083,10 +958,8 @@ if nodeinfo['packagesmaster']
         condition      = ${if <{$message_size}{256000}}
         spam           = pkg_user : true
         condition      = ${if >{$spam_score_int}{59}}
-'
-end
-out
-%>
+
+<% end %>
   accept
 
 
@@ -1124,21 +997,15 @@ begin routers
 #     An address is passed to each in turn until it is accepted.     #
 ######################################################################
 
-<%=
-out = ""
-if nodeinfo['mailrelay']
-  out = '
+<% if nodeinfo['mailrelay'] %>
 relay_manualroute:
   driver = manualroute
   domains = +mailhubdomains
   transport = remote_smtp
   route_data = ${lookup{$domain}lsearch{/etc/exim4/manualroute}}
   require_files = /etc/exim4/manualroute
-'
-end
-out
-%>
 
+<% end %>
 bsmtp:
   debug_print = "R: bsmtp for $local_part@$domain"
   driver = manualroute
@@ -1300,6 +1167,8 @@ procmail:
   check_local_user
   domains = +local_domains
   headers_add = "Delivered-To: ${local_part}${local_part_suffix}@${domain}"
+  local_part_suffix = -*
+  local_part_suffix_optional
   no_verify
   no_expn
   require_files = $local_part:$home/.procmailrc
@@ -1337,13 +1206,7 @@ localuser:
 # Everything before here should apply only to the local domains with a 
 # domains= rule
 
-# exim4 fails the router if it can't change to the user/group for delivery
-# during verification.  So we have to seperate the cases of verifying
-# the virts, and delivering to them.  blah.
-<%=
-out = ""
-if nodeinfo['packagesmaster']
-  out = '
+<% if nodeinfo['packagesmaster'] %>
 # This router delivers for packages.d.o
 packages:
   debug_print = "R: packages for $local_part@$domain"
@@ -1359,11 +1222,8 @@ packages:
   check_ancestor
   retry_use_local_part
   no_more
-'
-end
-out
-%>
 
+<% end %>
 <%=
 out = ""
 if nodeinfo['bugsmaster'] or nodeinfo['bugsmx']
@@ -1383,18 +1243,14 @@ bugs:
   no_more
   local_parts = ${if match\
                   {$local_part}\
-                  {\N^(\d+)(\d{2})(?:-(?:(?:submit|maintonly|quiet|forwarded|done|close|request|submitter)|(?:unsubscribe|ignore|(?:sub(?:scribe|help|yes|approve|reject))|unsubyes|bounce|probe|approve|reject|setlistyes|setlistsilentyes).*))?$\N}\
+                  {\N^(\d+)(\d{2})(?:-(?:(?:submit|maintonly|quiet|forwarded|done|close|request|submitter)|(?:unsubscribe|ignore|help|(?:sub(?:scribe|help|yes|approve|reject))|unsubyes|bounce|probe|approve|reject|setlistyes|setlistsilentyes).*))?$\N}\
                {${if exists{/org/bugs.debian.org/spool/db-h/$2/$1$2.summary}\
               {$local_part}fail}}fail}
 '
 end
 out
 %>
-
-<%=
-out = ""
-if nodeinfo['rtmaster']
-  out = '
+<% if nodeinfo['rtmaster'] %>
 # This router delivers for rt.d.o
 rt_force_new_verbose:
   debug_print = "R: rt for $local_part+new@$domain"
@@ -1404,14 +1260,14 @@ rt_force_new_verbose:
   local_parts = ${lookup{${sg{$local_part}{-comment}{}}}lsearch{RT_QUEUE_MAP}{$local_part}{}}
   local_part_suffix = +new
   pipe_transport = rt_pipe
-  data = "|/usr/bin/rt-mailgate --queue \'${lookup{${sg{$local_part}{-comment}{}}}lsearch{RT_QUEUE_MAP}}\' --url https://rt.debian.org/ --action ${if match{$local_part}{.*-comment.*}{comment}{correspond}}"
+  data = "|/usr/bin/rt-mailgate --queue '${lookup{${sg{$local_part}{-comment}{}}}lsearch{RT_QUEUE_MAP}}' --url https://rt.debian.org/ --action ${if match{$local_part}{.*-comment.*}{comment}{correspond}}"
   headers_add = "Delivered-To: ${local_part}${local_part_suffix}@${domain}"
 
 # FIXME: figure out how to generalize this approach so that all of the following would work
 # - rt+NNNN@rt.debian.org          : attach correspondence to ticket (verbose)
 # - rt+NNNN-quiesce@rt.debian.org  : attach correspondence to ticket (quiesce)
 # - rt+NNNN-<action>@rt.debian.org : attach correspondence to ticket (some action)
-# requires modification to custom condition in \'scrips\'
+# requires modification to custom condition in 'scrips'
 rt_force_new_quiesce:
   debug_print = "R: rt for $local_part+new-quiesce@$domain"
   driver = redirect
@@ -1420,7 +1276,7 @@ rt_force_new_quiesce:
   local_parts = ${lookup{${sg{$local_part}{-comment}{}}}lsearch{RT_QUEUE_MAP}{$local_part}{}}
   local_part_suffix = +new-quiesce
   pipe_transport = rt_pipe
-  data = "|/usr/bin/rt-mailgate --queue \'${lookup{${sg{$local_part}{-comment}{}}}lsearch{RT_QUEUE_MAP}}\' --url https://rt.debian.org/ --action ${if match{$local_part}{.*-comment.*}{comment}{correspond}}"
+  data = "|/usr/bin/rt-mailgate --queue '${lookup{${sg{$local_part}{-comment}{}}}lsearch{RT_QUEUE_MAP}}' --url https://rt.debian.org/ --action ${if match{$local_part}{.*-comment.*}{comment}{correspond}}"
   headers_add = "Delivered-To: ${local_part}${local_part_suffix}@${domain}\nX-RT-Mode: quiesce"
 
 rt_otherwise:
@@ -1432,12 +1288,14 @@ rt_otherwise:
   local_part_suffix = +*
   local_part_suffix_optional
   pipe_transport = rt_pipe
-  data = "|/usr/bin/rt-mailgate --queue \'${lookup{${sg{$local_part}{-comment}{}}}lsearch{RT_QUEUE_MAP}}\' --url https://rt.debian.org/ --extension ticket --action ${if match{$local_part}{.*-comment.*}{comment}{correspond}}"
+  data = "|/usr/bin/rt-mailgate --queue '${lookup{${sg{$local_part}{-comment}{}}}lsearch{RT_QUEUE_MAP}}' --url https://rt.debian.org/ --extension ticket --action ${if match{$local_part}{.*-comment.*}{comment}{correspond}}"
   headers_add = "Delivered-To: ${local_part}${local_part_suffix}@${domain}"
-'
-end
-out
-%>
+
+<% end %>
+
+# exim4 fails the router if it can't change to the user/group for delivery
+# during verification.  So we have to seperate the cases of verifying
+# the virts, and delivering to them.  blah.
 
 virt_direct_verify:
   debug_print = "R: virt_direct for $local_part@$domain"
@@ -1599,14 +1457,10 @@ remote_smtp:
   driver = smtp
   connect_timeout = 1m
   delay_after_cutoff = false
-<%=
-out = ""
-if has_variable?("exim_ssl_certs") && exim_ssl_certs == "true"
-  out = "  tls_certificate = /etc/exim4/ssl/thishost.crt
-  tls_privatekey = /etc/exim4/ssl/thishost.key"
-end
-out
-%>
+<% if has_variable?("exim_ssl_certs") && exim_ssl_certs == "true" %>
+  tls_certificate = /etc/exim4/ssl/thishost.crt
+  tls_privatekey = /etc/exim4/ssl/thishost.key
+<% end %>
 
 <%=
 out = ""
@@ -1628,7 +1482,6 @@ remote_smtp_smarthost:
 end
 out
 %>
-
 # Send the message to procmail
 procmail_pipe:
   driver = pipe
@@ -1648,10 +1501,7 @@ bsmtp:
                     {$value}fail}\
                   }}
 
-<%=
-out = ""
-if nodeinfo['bugsmaster'] or nodeinfo['bugsmx']
-  out = '
+<% if nodeinfo['bugsmaster'] or nodeinfo['bugsmx'] %>
 bugs_pipe:
   driver = pipe
   command = /org/bugs.debian.org/mail/run-procmail
@@ -1662,25 +1512,17 @@ bugs_pipe:
   return_path_add
   return_output
   user = debbugs
-'
-end
-out
-%>
 
-<%=
-out = ""
-if nodeinfo['rtmaster']
-  out = '
+<% end %>
+<% if nodeinfo['rtmaster'] %>
 rt_pipe:
   debug_print = "T: rt_pipe for $local_part${local_part_suffix}@$domain"
   driver = pipe
   return_fail_output
   environment = EXTENSION=${substr_1:${local_part_suffix}}
   allow_commands = /usr/bin/rt-mailgate
-'
-end
-out
-%>
+
+<% end %>
 
 ######################################################################
 #                      RETRY CONFIGURATION                           #
@@ -1699,6 +1541,7 @@ out
 begin retry
 
 debian.org            *           F,2h,10m; G,16h,2h,1.5; F,14d,8h
+*                      * senders=: F,2h,10m
 *                      rcpt_4xx    F,2h,5m;  F,4h,10m; F,4d,15m
 *                      *           F,2h,15m; G,16h,2h,1.5; F,4d,8h