eximconf: simplify a sender domain check by using sender_domains

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

eximconf: log a few more details for messages

These make it easier to tie up incoming and outbound log entries
for a given mail.

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

Merge branch 'salsa' into fordsa

mekeel now gets the UBC autofs config

rename BM mekeel volumes to OLD-

Add volumes for mekeel

eximconf: add debug messages for (remaining) transports

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

eximconf: add a debug message for relay_manualroute

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

fail2ban: make exim "AUTH LOGIN" match case-insensitive

Apparently people are trying it in lower-case as well

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

fail2ban: add more expressions to the Exim filter

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

eximconf: increase log detail

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

eximconf: split log_selector to one item per line

This makes it easier to add/remove items

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

eximconf: force mail to Google to be routed via IPv4

Delivering mail to Google over IPv6 is tricky unless the stars align
in precisely the right manner. Doing so over IPv4 can still be
awkard, but is generally much simpler.

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

eximconf: make NDRs contain only headers, not bodies

This helps avoid us being used as a potential part of redirected
"bounce spam".

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

eximconf: add an alternative SMTP router for "single domain" domains

These are domains that only allow a single recipient domain per SMTP
connection, in order to allow simple differentiation of filtering
options.

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

Merge branch 'salsa' into fordsa

exim: fix syntax error in exim_surbl.pl

exim: remove debconf hosts from debianhosts

debconf.org email goes through the debian MXs now.

Merge branch 'fordsa' of https://git.adam-barratt.org.uk/git/mirror/dsa-puppet

Fix package name

And remove the file on or after 10.2

Ignore local package trapperkeeper-webserver-jetty9-clojure on puppetmaster, cf. Debian#924005, Debian#930562

eximconf: add submission-domains to file list

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

eximconf: remove explicit inclusion of 82.195.75.76

Was apparently once a DebConf host but is now a standard d.o host

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

eximconf: remove inaccurate comment

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

eximconf: use a more idiomatic way of "resetting" a variable

An empty string is as valid as any other, so there's no need for hacks.

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

eximconf: add notes on files sourced from UD

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

exim_surbl: disable DBL lookups

Probably shouldn't be enabled without some discussion

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

eximconf: drop etch-specific comment

It's a little obsolete by now

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

exim/manifests/init.pp: fix typo

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

eximconf.erb: simplify bugs.d.o router

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

exim: update {two,three}-level-tlds from SURBL

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

exim: ship new {two,three}-level-tlds files for exim_surbl.pl

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

exim: update exim_surbl.pl to version 2.3

- re-add Puppet headers
- fix file paths to use Debian's /etc/exim4
- add new {two,three}-level-tlds files

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

eximconf.erb: macroize maximum content scanning message size

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

eximconf.erb: standardise on style of condition checks

From the Exim documentation:
"If both strings are omitted, the result is the string true if the
condition is true, and the empty string if the condition is false.
This makes it less cumbersome to write custom ACL and router
conditions."

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

eximconf.erb: typo fix ("seperate" -> "separate")

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

eximconf.erb: consistently capitalise Exim and Debian

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

eximconf.erb: fix typo ("usefull" -> "useful")

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

eximconf.erb: fix address mentioned in postmaster@d.o check

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

eximconf.erb: fix some typoes

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

eximconf: correct obsolete references to "/etc/exim"

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

Remove no longer required submission-domains override for busoni

busoni itself no longer exists and the bugs.d.o submission override
is now handled differently.

Signed-off-by: Adam D. Barratt <adam@adam-barratt.org.uk>

remove non-standard From header from Bacula emails

re Debian#935886, by anarcat

simplify email_error default logic

(by anarcat)

Fix bacula email logic

Revert "Since the conditional does not work for default values for params, fall back to sane defaults in code"

This reverts commit 75ce99c03293e2df071bcbb7f56694d192652586.

Since the conditional does not work for default values for params, fall back to sane defaults in code

set email_error until we figure out how to fix it properly

Only realize the director-from-client config if the director-from-client-via-storage config is already there

Split up mail roles

Try to wait for both network-online *and* unbound

defaults for db_address, db_port; move dbsslmode = verify-ca into the case where we have a dbsslca

List non-optional params first

Move bacula::bacula_ssl_{server,client}_{cert,key} to hiera

Move bacula::bacula_ssl_ca_path to hiera

s/bacula_ca_path/bacula_ssl_ca_path/

Move operator_email to hiera

bacula_dsa_client_list and tag_bacula_dsa_client_list are not parameters

Document ferm::rule::simple

dsa_systemd::override: cleanup and make sure removal cleans the dir also

Make rentention periods configurable with hiera, per client

Globally enable Heartbeat Interval

Remove defaults

Unify name of the Messages Resource

Try after unbound.service instead of network-online.target

Doing the after network-online.target was not sufficient.

: Config error: Cannot add hostname(adayevskaya.debian.org) and port(9102) to addrlist (Cannot resolve hostname(adayevskaya.debian.org) Temporary failure in name resolution)
:             : line 23, col 5 of file /etc/bacula/bacula-fd.conf
:     }

All our systems are now at least Debian 9

sd and fd look up hostnames, so start them only after the network is online -a

Use a variable that still exists to define the listen hostname

fd: also use the grep in public_addresses to learn if we support v4/v6

Pass pool-name token to volumes-delete-old

Try to collect directors before we restart fd

Only restart once we have at least one file in the .d dir

another notify

Unnecessary requires

And correct location and template for dir snippet on fd

Correct notify for dir snippet on fd

Correct empty.conf location for fd

New default dir name

Create an empty file in the FD's conf.d dir

I do not think empty directories need a source => (these days)

Get director name from director

move director db pw to a more local thing

Get the ssl variables from bacula:: as we have not inherited them

continue with moving director address to a more local thing

continue with moving director name/address to a more local thing

retire unused var

Move ipv[46] check into the manifest

Make a few Strings be Stdlib::Host

Remove bacula-monitor.  This seems unused for now, but we can always bring it back if we think it necessary

start with moving director name to a more local thing

And storage name is sd local

And pass the right variable

and use new variable in template

Create client name and secret in the fd and ship

and use fqdn in hkdf

Make director_secret for bconsole something local to the director class

Make bacula_director_port something local to the director class

explicitly pass client name around

Give the RestoreFiles Job needs a pool.  any pool.