docs: Revise `bind-smtp-network-interface.md`

.github/workflows/generic_build.yml

@@ -71,16 +71,16 @@ jobs:
             cache-buildx-
 
       - name: 'Set up QEMU'
-        uses: docker/setup-qemu-action@v3.6.0
+        uses: docker/setup-qemu-action@v3.3.0
         with:
           platforms: arm64
 
       - name: 'Set up Docker Buildx'
-        uses: docker/setup-buildx-action@v3.10.0
+        uses: docker/setup-buildx-action@v3.8.0
 
       # NOTE: AMD64 can build within 2 minutes
       - name: 'Build images'
-        uses: docker/build-push-action@v6.15.0
+        uses: docker/build-push-action@v6.13.0
         with:
           context: .
           # Build at least the AMD64 image (which runs against the test suite).

.github/workflows/generic_publish.yml

@@ -23,7 +23,7 @@ jobs:
 
       - name: 'Prepare tags'
         id: prep
-        uses: docker/metadata-action@v5.7.0
+        uses: docker/metadata-action@v5.6.1
         with:
           images: |
             ${{ secrets.DOCKER_REPOSITORY }}
@@ -35,12 +35,12 @@ jobs:
             type=semver,pattern={{major}}.{{minor}}.{{patch}}
 
       - name: 'Set up QEMU'
-        uses: docker/setup-qemu-action@v3.6.0
+        uses: docker/setup-qemu-action@v3.3.0
         with:
           platforms: arm64
 
       - name: 'Set up Docker Buildx'
-        uses: docker/setup-buildx-action@v3.10.0
+        uses: docker/setup-buildx-action@v3.8.0
 
       # Try get the cached build layers from a prior `generic_build.yml` job.
       # NOTE: Until adopting `type=gha` scoped cache exporter (in `docker/build-push-action`),
@@ -67,7 +67,7 @@ jobs:
           password: ${{ secrets.GITHUB_TOKEN }}
 
       - name: 'Build and publish images'
-        uses: docker/build-push-action@v6.15.0
+        uses: docker/build-push-action@v6.13.0
         with:
           context: .
           build-args: |

.github/workflows/generic_test.yml

@@ -38,12 +38,12 @@ jobs:
       # Ensures consistent BuildKit version (not coupled to Docker Engine),
       # and increased compatibility of the build cache vs mixing buildx drivers.
       - name: 'Set up Docker Buildx'
-        uses: docker/setup-buildx-action@v3.10.0
+        uses: docker/setup-buildx-action@v3.8.0
 
       # Importing from the cache should create the image within approx 30 seconds:
       # NOTE: `qemu` step is not needed as we only test for AMD64.
       - name: 'Build AMD64 image from cache'
-        uses: docker/build-push-action@v6.15.0
+        uses: docker/build-push-action@v6.13.0
         with:
           context: .
           tags: mailserver-testing:ci

.github/workflows/generic_vulnerability-scan.yml

@@ -37,12 +37,12 @@ jobs:
       # Ensures consistent BuildKit version (not coupled to Docker Engine),
       # and increased compatibility of the build cache vs mixing buildx drivers.
       - name: 'Set up Docker Buildx'
-        uses: docker/setup-buildx-action@v3.10.0
+        uses: docker/setup-buildx-action@v3.8.0
 
       # Importing from the cache should create the image within approx 30 seconds:
       # NOTE: `qemu` step is not needed as we only test for AMD64.
       - name: 'Build AMD64 image from cache'
-        uses: docker/build-push-action@v6.15.0
+        uses: docker/build-push-action@v6.13.0
         with:
           context: .
           tags: mailserver-testing:ci

.github/workflows/scripts/docs/build-docs.sh

@@ -11,7 +11,7 @@ docker run \
   --user "$(id -u):$(id -g)" \
   --volume "./:/docs" \
   --name "build-docs" \
-  squidfunk/mkdocs-material:9.6 build --strict
+  squidfunk/mkdocs-material:9.5 build --strict
 
 # Remove unnecessary build artifacts: https://github.com/squidfunk/mkdocs-material/issues/2519
 # site/ is the build output folder.

CHANGELOG.md

@@ -2,107 +2,62 @@
 
 All notable changes to this project will be documented in this file. The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/), and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [Unreleased](https://github.com/docker-mailserver/docker-mailserver/compare/v15.0.2...HEAD)
+## [Unreleased](https://github.com/docker-mailserver/docker-mailserver/compare/v14.0.0...HEAD)
 
 > **Note**: Changes and additions listed here are contained in the `:edge` image tag. These changes may not be as stable as released changes.
 
-### Updates
-
-- **Documentation:**
-  - Added a compatibility note for a Dovecot + Solr 9.8 breaking change ([#4433](https://github.com/docker-mailserver/docker-mailserver/pull/4433))
-- **Internal:**
-  - Refactored `setup config dkim` (`open-dkim`) ([#4375](https://github.com/docker-mailserver/docker-mailserver/pull/4375))
-
-## [v15.0.2](https://github.com/docker-mailserver/docker-mailserver/releases/tag/v15.0.2)
-
-### Fixes
-
-- **Postfix**
-  - Avoid modifying the message body when filtering sender headers. This regression was introduced from [#4120](https://github.com/docker-mailserver/docker-mailserver/pull/4120) as part of DMS v15.0.0 ([#4429](https://github.com/docker-mailserver/docker-mailserver/pull/4429))
-
-## [v15.0.1](https://github.com/docker-mailserver/docker-mailserver/releases/tag/v15.0.1)
-
-### Added
-
-- **Internal:**
-  - Added the Smallstep `step` CLI command for future internal usage ([#4376](https://github.com/docker-mailserver/docker-mailserver/pull/4376))
-
-### Fixes
-
-- **Postfix:**
-  - `setup email restrict` generated configs now only prepend to `dms_smtpd_sender_restrictions` ([#4379](https://github.com/docker-mailserver/docker-mailserver/pull/4379))
-- **Rspamd:**
-  - Change detection support now monitors all files found within the DMS _Config Volume_ Rspamd directory ([#4418](https://github.com/docker-mailserver/docker-mailserver/pull/4418))
-- **Internal:**
-  - A permissions fix for `/var/log/mail` that was [added in DMS v15]((https://github.com/docker-mailserver/docker-mailserver/pull/4374)) no longer encounters an error when no log files are present during a container restart, such as with a `tmpfs` volume mount ([#4391](https://github.com/docker-mailserver/docker-mailserver/pull/4391))
-  - The DMS _State Volume_ (`/var/mail-state`) will now ensure it's file tree is accessible for services when the volume was created with missing executable bit ([#4420](https://github.com/docker-mailserver/docker-mailserver/pull/4420))
-  - The DMS _Config Volume_ (`/tmp/docker-mailserver`) now correctly updates permissions on container restarts ([#4417](https://github.com/docker-mailserver/docker-mailserver/pull/4417))
-
-### Updates
-
-- **Internal:**
-  - Minor improvements to `_install_utils()` in `packages.sh` ([#4376](https://github.com/docker-mailserver/docker-mailserver/pull/4376))
-
-## [v15.0.0](https://github.com/docker-mailserver/docker-mailserver/releases/tag/v15.0.0)
-
 ### Breaking
 
 - **saslauthd** mechanism support via ENV `SASLAUTHD_MECHANISMS` with `pam`, `shadow`, `mysql` values has been removed. Only `ldap` and `rimap` remain supported ([#4259](https://github.com/docker-mailserver/docker-mailserver/pull/4259))
 - **getmail6** has been refactored: ([#4156](https://github.com/docker-mailserver/docker-mailserver/pull/4156))
   - The [DMS config volume](https://docker-mailserver.github.io/docker-mailserver/v15.0/config/advanced/optional-config/#volumes) now has support for `getmailrc_general.cf` for overriding [common default settings](https://docker-mailserver.github.io/docker-mailserver/v15.0/config/advanced/mail-getmail/#common-options). If you previously mounted this config file directly to `/etc/getmailrc_general` you should switch to our config volume support.
+  - IMAP/POP3 example configs added to our [`config-examples`](https://github.com/docker-mailserver/docker-mailserver/tree/v15.0.0/config-examples/getmail).
+  - ENV [`GETMAIL_POLL`](https://docker-mailserver.github.io/docker-mailserver/v15.0/config/environment/#getmail_poll) now supports values above 30 minutes.
+  - Added `getmail` as a new service for `supervisor` to manage, replacing cron for periodic polling.
   - Generated getmail configuration files no longer set the `message_log` option. Instead of individual log files per config, the [default base settings DMS configures](https://github.com/docker-mailserver/docker-mailserver/tree/v15.0.0/target/getmail/getmailrc_general) now enables `message_log_syslog`. This aligns with how other services in DMS log to syslog where it is captured in `mail.log`.
   - Getmail configurations have changed location from the base of the DMS Config Volume, to the `getmail/` subdirectory. Any existing configurations **must be migrated manually.**
-  - **DMS v14 mistakenly** relocated the _getmail state directory_ to the _DMS Config Volume_ as a `getmail/` subdirectory.
+  - DMS v14 mistakenly relocated the _getmail state directory_ to the _DMS Config Volume_ as a `getmail/` subdirectory.
     - This has been corrected to `/var/lib/getmail` (_if you have mounted a DMS State Volume to `/var/mail-state`, `/var/lib/getmail` will be symlinked to `/var/mail-state/lib-getmail`_).
     - To preserve this state when upgrading to DMS v15, **you must manually migrate `getmail/` from the _DMS Config Volume_ to `lib-getmail/` in the _DMS State Volume_.**
-  - `setup email delete <EMAIL ADDRESS>` now requires explicit confirmation if the mailbox data should be deleted ([#4365](https://github.com/docker-mailserver/docker-mailserver/pull/4365)).
+- **removed `VERSION`** file that was used for checking version updates ([#3677](https://github.com/docker-mailserver/docker-mailserver/issues/3677),[#4321](https://github.com/docker-mailserver/docker-mailserver/pull/4321))
-- **Rspamd:** Removed deprecated file path check (_DMS config volume: `./rspamd-modules.conf` => `./rspamd/custom-commands.conf`_) ([#4373](https://github.com/docker-mailserver/docker-mailserver/pull/4373))
+
+### Security
+
+- **Fail2ban:**
+  - Ensure a secure connection, when downloading the fail2ban package ([#4080](https://github.com/docker-mailserver/docker-mailserver/pull/4080))
 
 ### Added
 
 - **Internal:**
   - Add password confirmation to several `setup` CLI subcommands ([#4072](https://github.com/docker-mailserver/docker-mailserver/pull/4072))
-  - Added a `debug getmail` subcommand to `setup` ([#4346](https://github.com/docker-mailserver/docker-mailserver/pull/4346))
 
 ### Updates
 
-- **Internal:**
+- **Fail2ban:**
-  - **Removed `VERSION` file** from the repo. Releases of DMS prior to v13 (Nov 2023) would check this to detect new releases ([#3677](https://github.com/docker-mailserver/docker-mailserver/issues/3677), [#4321](https://github.com/docker-mailserver/docker-mailserver/pull/4321))
+  - Updated to version [`1.1.0`](https://github.com/fail2ban/fail2ban/releases/tag/1.1.0) ([#4045](https://github.com/docker-mailserver/docker-mailserver/pull/4045))
-  - During image build, ensure a secure connection when downloading the `fail2ban` package ([#4080](https://github.com/docker-mailserver/docker-mailserver/pull/4080))
 - **Documentation:**
   - Account Management and Authentication pages have been rewritten and better organized ([#4122](https://github.com/docker-mailserver/docker-mailserver/pull/4122))
   - Add a caveat for `DMS_VMAIL_UID` not being compatible with `0` / root ([#4143](https://github.com/docker-mailserver/docker-mailserver/pull/4143))
-- **Getmail:** ([#4156](https://github.com/docker-mailserver/docker-mailserver/pull/4156))
-  - Added `getmail` as a new service for `supervisor` to manage, replacing cron for periodic polling.
-  - IMAP/POP3 example configs added to our [`config-examples`](https://github.com/docker-mailserver/docker-mailserver/tree/v15.0.0/config-examples/getmail).
-  - ENV [`GETMAIL_POLL`](https://docker-mailserver.github.io/docker-mailserver/v15.0/config/environment/#getmail_poll) now supports values above 30 minutes.
 - **Postfix:**
   - By default opt-out from _Microsoft reactions_ for outbound mail ([#4120](https://github.com/docker-mailserver/docker-mailserver/pull/4120))
-- **Rspamd:**
+- Updated `jaq` version from `1.3.0` to `2.0.0` ([#4190](https://github.com/docker-mailserver/docker-mailserver/pull/4190))
-  - Updated GTube settings and tests ([#4191](https://github.com/docker-mailserver/docker-mailserver/pull/4191))
+- Updated Rspamd GTube settings and tests ([#4191](https://github.com/docker-mailserver/docker-mailserver/pull/4191))
-- Updated externally installed software ([#4357](https://github.com/docker-mailserver/docker-mailserver/pull/4357)):
-  - `DOVECOT_COMMUNITY_REPO=1` custom image build ARG now supports the latest Dovecot [`2.4.x`](https://github.com/dovecot/core/releases/tag/2.4.0) (_DMS provides Dovecot `2.3.19` by default_)
-  - Dovecot FTS Xapian module (`1.7.12` => [`1.9.0`](https://github.com/grosjo/fts-xapian/releases/tag/1.9))
-  - `jaq` (`1.3.0` => [`2.1.0`](https://github.com/01mf02/jaq/releases/tag/v2.1.0))
-  - Fail2Ban (`1.0.2-2` => [`1.1.0`](https://github.com/fail2ban/fail2ban/releases/tag/1.1.0)) ([#4045](https://github.com/docker-mailserver/docker-mailserver/pull/4045))
-  - Rspamd (`3.8.4` => [`3.11.0`](https://github.com/rspamd/rspamd/releases/tag/3.11.0)) - Implicitly upgraded during image build, as the third-party repo lacks version pinning support.
 
 ### Fixes
 
 - **Dovecot:**
   - The logwatch `ignore.conf` now also excludes Xapian messages about pending documents ([#4060](https://github.com/docker-mailserver/docker-mailserver/pull/4060))
-  - `dovecot-fts-xapian` plugin was updated, fixing a regression with indexing ([#4095](https://github.com/docker-mailserver/docker-mailserver/pull/4095))
+  - `dovecot-fts-xapian` plugin was updated to `1.7.13`, fixing a regression with indexing ([#4095](https://github.com/docker-mailserver/docker-mailserver/pull/4095))
   - The "dummy account" workaround for _Dovecot Quota_ feature support no longer treats the alias as a regex when checking the Dovecot UserDB ([#4222](https://github.com/docker-mailserver/docker-mailserver/pull/4222))
 - **LDAP:**
-  - Correctly apply a compatibility fix for OAuth2 introduced in DMS `v13.3.1` which had not been applied to the actual LDAP config changes ([#4175](https://github.com/docker-mailserver/docker-mailserver/pull/4175))
+  - Correctly apply a compatibility fix for OAuth2 introduced in DMS v13.3.1 which had not been applied to the actual LDAP config changes ([#4175](https://github.com/docker-mailserver/docker-mailserver/pull/4175))
 - **Internal:**
   - The main `mail.log` (_which is piped to stdout via `tail`_) now correctly begins from the first log line of the active container run. Previously some daemon logs and potential warnings/errors were omitted ([#4146](https://github.com/docker-mailserver/docker-mailserver/pull/4146))
+  - Fixed a regression introduced in v14 where `postfix-main.cf` appended `stderr` output into `/etc/postfix/main.cf`, causing Postfix startup to fail ([#4147](https://github.com/docker-mailserver/docker-mailserver/pull/4147))
   - `start-mailserver.sh` removed unused `shopt -s inherit_errexit` ([#4161](https://github.com/docker-mailserver/docker-mailserver/pull/4161))
-  - Fixed a regression introduced in DMS v14 where `postfix-main.cf` appended `stderr` output into `/etc/postfix/main.cf`, causing Postfix startup to fail ([#4147](https://github.com/docker-mailserver/docker-mailserver/pull/4147))
-  - Fixed a regression introduced in DMS v14 to better support running `start-mailserver.sh` with container restarts, which now only skip calling `_setup()` ([#4323](https://github.com/docker-mailserver/docker-mailserver/pull/4323#issuecomment-2629559254), [#4374](https://github.com/docker-mailserver/docker-mailserver/pull/4374))
-  - The command `swaks --help` is now functional ([#4282](https://github.com/docker-mailserver/docker-mailserver/pull/4282))
 - **Rspamd:**
   - DKIM private key path checking is now performed only on paths that do not contain `$` ([#4201](https://github.com/docker-mailserver/docker-mailserver/pull/4201))
+- The command `swaks --help` is now functional ([#4282](https://github.com/docker-mailserver/docker-mailserver/pull/4282))
 
 ### CI
 

Dockerfile

@@ -88,6 +88,16 @@ RUN dpkg -i /dovecot-fts-xapian-*.deb && rm /dovecot-fts-xapian-*.deb
 COPY target/dovecot/*.inc target/dovecot/*.conf /etc/dovecot/conf.d/
 COPY target/dovecot/dovecot-purge.cron /etc/cron.d/dovecot-purge.disabled
 RUN chmod 0 /etc/cron.d/dovecot-purge.disabled
+WORKDIR /usr/share/dovecot
+
+# hadolint ignore=SC2016,SC2086,SC2069
+RUN <<EOF
+  sedfile -i -e 's/include_try \/usr\/share\/dovecot\/protocols\.d/include_try \/etc\/dovecot\/protocols\.d/g' /etc/dovecot/dovecot.conf
+  sedfile -i -e 's/#mail_plugins = \$mail_plugins/mail_plugins = \$mail_plugins sieve/g' /etc/dovecot/conf.d/15-lda.conf
+  sedfile -i -e 's/^.*lda_mailbox_autocreate.*/lda_mailbox_autocreate = yes/g' /etc/dovecot/conf.d/15-lda.conf
+  sedfile -i -e 's/^.*lda_mailbox_autosubscribe.*/lda_mailbox_autosubscribe = yes/g' /etc/dovecot/conf.d/15-lda.conf
+  sedfile -i -e 's/^.*postmaster_address.*/postmaster_address = '${POSTMASTER_ADDRESS:="postmaster@domain.com"}'/g' /etc/dovecot/conf.d/15-lda.conf
+EOF
 
 # -----------------------------------------------
 # --- Rspamd ------------------------------------

demo-setups/fetchmail-compose.yaml

@@ -1,10 +1,10 @@
-# Docs: https://docker-mailserver.github.io/docker-mailserver/v15.0/config/advanced/mail-fetchmail
+# Docs: https://docker-mailserver.github.io/docker-mailserver/v14.0/config/advanced/mail-fetchmail
 # Additional context, with CLI commands for verification:
 # https://github.com/orgs/docker-mailserver/discussions/3994#discussioncomment-9290570
 
 services:
   dms-fetch:
-    image: ghcr.io/docker-mailserver/docker-mailserver:latest # :15.0
+    image: ghcr.io/docker-mailserver/docker-mailserver:latest # :14.0
     hostname: mail.example.test
     environment:
       ENABLE_FETCHMAIL: 1
@@ -26,7 +26,7 @@ services:
         target: /tmp/docker-mailserver/fetchmail.cf
 
   dms-remote:
-    image: ghcr.io/docker-mailserver/docker-mailserver:latest # :15.0
+    image: ghcr.io/docker-mailserver/docker-mailserver:latest # :14.0
     hostname: mail.remote.test
     environment:
       # Allows for us send a test mail easily by trusting any mail client run within this container (`swaks`):

demo-setups/relay-compose.yaml

@@ -1,11 +1,11 @@
-# Docs: https://docker-mailserver.github.io/docker-mailserver/v15.0/config/advanced/mail-forwarding/relay-hosts/
+# Docs: https://docker-mailserver.github.io/docker-mailserver/v14.0/config/advanced/mail-forwarding/relay-hosts/
 # Additional context, with CLI commands for verification:
 # https://github.com/docker-mailserver/docker-mailserver/issues/4136#issuecomment-2253693490
 
 services:
   # This would represent your actual DMS container:
   dms-sender:
-    image: mailserver/docker-mailserver:latest # :15.0
+    image: mailserver/docker-mailserver:latest # :14.0
     hostname: mail.example.test
     environment:
       # All outbound mail will be relayed through this host
@@ -37,17 +37,21 @@ services:
 
   # Pretend this is your third-party relay service:
   dms-relay:
-    image: mailserver/docker-mailserver:latest # :15.0
+    image: mailserver/docker-mailserver:latest # :14.0
     hostname: smtp.relay-service.test
     environment:
       # WORKAROUND: Bypass security checks from the mail-client (dms-sender container)
-      # (avoids needing expected DNS records to run this example)
+      # (avoids needing valid DNS for this example)
       - PERMIT_DOCKER=connected-networks
       # TLS is required when relaying to dms-relay via ports 587 / 465
       # (dms-relay will then relay the mail to dms-destination over port 25)
       - SSL_TYPE=manual
       - SSL_KEY_PATH=/tmp/tls/key.pem
       - SSL_CERT_PATH=/tmp/tls/cert.pem
+    # WORKAROUND: `links` is required due to lack of properly configured DNS.
+    # (resolves destination.test to the IP of the dms-destination container)
+    links:
+      - "dms-destination:destination.test"
     configs:
       - source: dms-accounts-relay
         target: /tmp/docker-mailserver/postfix-accounts.cf
@@ -58,16 +62,10 @@ services:
 
   # Pretend this is another mail server that your target recipient belongs to (like Gmail):
   dms-destination:
-    image: mailserver/docker-mailserver:latest # :15.0
+    image: mailserver/docker-mailserver:latest # :14.0
     hostname: mail.destination.test
-    # WORKAROUND: dms-relay must be able to resolve DNS for `@destination.test` to the IP of this container:
+    # Same workaround for purposes of the example, with the target recipient provisioned to accept mail
-    # Normally a MX record would direct mail to the MTA (eg: `mail.destination.test`)
-    networks:
-      default:
-        aliases:
-          - destination.test
     environment:
-      # WORKAROUND: Same workaround as needed for dms-relay
       - PERMIT_DOCKER=connected-networks
     configs:
       - source: dms-accounts-destination
@@ -77,13 +75,6 @@ services:
 # NOTE: This feature requires Docker Compose v2.23.1 (Nov 2023) or newer:
 # https://github.com/compose-spec/compose-spec/pull/446
 configs:
-  # `postfix-main.cf`, a single line change to make all outbound SMTP connections over implicit TLS instead of the default explicit TLS (StartTLS).
-  # NOTE: If you need to only selectively relay mail, you would need to instead adjust this on the relay service in `/etc/postfix/master.cf`,
-  # However DMS presently modifies this when using the DMS Relay Host feature support, which may override `postfix-master.cf` or `user-patches.sh` due to `check-for-changes.sh`.
-  dms-main:
-    content: |
-      smtp_tls_wrappermode=yes
-
   # DMS expects an account to be configured to run, this example provides accounts already created.
   # Login credentials:
   # user: "john.doe@example.test" password: "secret"
@@ -103,6 +94,13 @@ configs:
     content: |
       jane.doe@destination.test|{SHA512-CRYPT}$$6$$o65y1ZXC4ooOPLwZ$$7TF1nYowEtNJpH6BwJBgdj2pPAxaCvhIKQA6ww5zdHm/AA7aemY9eoHC91DOgYNaKj1HLxSeWNDdvrp6mbtUY.
 
+  # This is `postfix-main.cf`, single line change to make all outbound SMTP connections over port 465 instead of 25 (default)
+  # If you selectively relay mail, you would need to adjust this on the relay service in `/etc/postfix/master.cf`,
+  # However DMS presently modifies this when using the DMS Relay Host feature support, which may override `postfix-master.cf` or `user-patches.sh` due to `check-for-changes.sh`.
+  dms-main:
+    content: |
+      smtp_tls_wrappermode=yes
+
   # TLS files:
   # - Use an ECDSA cert that's been signed by a self-signed CA for TLS cert verification.
   # - This cert is only valid for mail.example.test, mail.destination.test, smtp.relay-service.test

docs/content/config/advanced/mail-forwarding/relay-hosts.md

@@ -151,6 +151,6 @@ We provide this support via two config files:
 [wikipedia::smarthost]: https://en.wikipedia.org/wiki/Smart_host
 
 [docs::env-relay]: ../../environment.md#relay-host
-[dms-repo::helpers-relay]: https://github.com/docker-mailserver/docker-mailserver/blob/v15.0.0/target/scripts/helpers/relay.sh
+[dms-repo::helpers-relay]: https://github.com/docker-mailserver/docker-mailserver/blob/v14.0.0/target/scripts/helpers/relay.sh
 [dms-gh::pr-3607]: https://github.com/docker-mailserver/docker-mailserver/issues/3607
 [dms-gh::relay-example]: https://github.com/docker-mailserver/docker-mailserver/issues/3842#issuecomment-1913380639

docs/content/config/advanced/mail-getmail.md

@@ -108,11 +108,3 @@ It is possible to utilize the `getmail-gmail-xoauth-tokens` helper to provide au
 [getmail-docs]: https://getmail6.org/configuration.html
 [getmail-docs-xoauth-12]: https://github.com/getmail6/getmail6/blob/1f95606156231f1e074ba62a9baa64f892a92ef8/docs/getmailrc-examples#L286
 [getmail-docs-xoauth-13]: https://github.com/getmail6/getmail6/blob/1f95606156231f1e074ba62a9baa64f892a92ef8/docs/getmailrc-examples#L351
-
-## Debugging
-
-To debug your `getmail` configurations, run this `setup debug` command:
-
-```sh
-docker exec -it dms-container-name setup debug getmail
-```

docs/content/config/best-practices/dkim_dmarc_spf.md

@@ -29,7 +29,7 @@ Cloudflare has written an [article about DKIM, DMARC and SPF][cloudflare-dkim-dm
 When DKIM is enabled:
 
 1. Inbound mail will verify any included DKIM signatures
-2. Outbound mail is signed (_when your sending domain has a configured DKIM key_)
+2. Outbound mail is signed (_when you're sending domain has a configured DKIM key_)
 
 DKIM requires a public/private key pair to enable **signing (_via private key_)** your outgoing mail, while the receiving end must query DNS to **verify (_via public key_)** that the signature is trustworthy.
 
@@ -54,7 +54,7 @@ You'll need to repeat this process if you add any new domains.
 
 You should have:
 
-- At least one [email account setup][docs-accounts]
+- At least one [email account setup][docs-accounts-add]
 - Attached a [volume for config][docs-volumes-config] to persist the generated files to local storage
 
 !!! example "Creating DKIM Keys"

docs/content/config/debugging.md

@@ -111,7 +111,7 @@ This could be from outdated software, or running a system that isn't able to pro
 - **Container runtime:** Docker and Podman for example have subtle differences. DMS docs are primarily focused on Docker, but we try to document known issues where relevant.
 - **Rootless containers:** Introduces additional differences in behavior or requirements:
     - cgroup v2 is required for supporting rootless containers.
-    - Differences such as for container networking which may further affect support for IPv6 and preserving the client IP (Remote address). Example with Docker rootless are [binding a port to a specific interface][docker-rootless-interface] and the choice of [port forwarding driver][docs::fail2ban::rootless-portdriver].
+    - Differences such as for container networking which may further affect support for IPv6 and preserving the client IP (Remote address). Example with Docker rootless are [binding a port to a specific interface][docker-rootless-interface] and the choice of [port forwarding driver][docs-rootless-portdriver].
 
 [network::docker-userlandproxy]: https://github.com/moby/moby/issues/44721
 [network::docker-nftables]: https://github.com/moby/moby/issues/26824

docs/content/config/environment.md

@@ -202,7 +202,7 @@ Please read [the SSL page in the documentation][docs-tls] for more information.
 Configures the handling of creating mails with forged sender addresses.
 
 - **0** => (not recommended) Mail address spoofing allowed. Any logged in user may create email messages with a [forged sender address](https://en.wikipedia.org/wiki/Email_spoofing).
-- 1 => Mail spoofing denied. Each user may only send with their own or their alias addresses. Addresses with [extension delimiters](http://www.postfix.org/postconf.5.html#recipient_delimiter) are not able to send messages.
+- 1 => Mail spoofing denied. Each user may only send with his own or his alias addresses. Addresses with [extension delimiters](http://www.postfix.org/postconf.5.html#recipient_delimiter) are not able to send messages.
 
 ##### ENABLE_SRS
 

docs/content/config/security/fail2ban.md

@@ -14,48 +14,18 @@ hide:
 
 ## Configuration
 
-Enabling Fail2Ban support can be done via ENV, but also requires granting at least the `NET_ADMIN` capability to interact with the kernel and ban IP addresses.
+!!! warning
 
-!!! example
+    DMS must be launched with the `NET_ADMIN` capability in order to be able to install the NFTables rules that actually ban IP addresses. Thus, either include `--cap-add=NET_ADMIN` in the `docker run` command, or the equivalent in the `compose.yaml`:
 
-    === "Docker Compose"
+    ```yaml
-
+    cap_add:
-        ```yaml title="compose.yaml"
+      - NET_ADMIN
-        services:
+    ```
-          mailserver:
-            environment:
-              - ENABLE_FAIL2BAN=1
-            cap_add:
-              - NET_ADMIN
-        ```
-
-    === "Docker CLI"
-
-        ```bash
-        docker run --rm -it \
-          --cap-add=NET_ADMIN \
-          --env ENABLE_FAIL2BAN=1
-        ```
-
-!!! warning "Security risk of adding non-default capabilties"
-
-    DMS bundles F2B into the image for convenience to simplify integration and deployment.
-
-    The [`NET_ADMIN`][security::cap-net-admin] and [`NET_RAW`][security::cap-net-raw] capabilities are not granted by default to the container root user, as they can be used to compromise security.
-
-    If this risk concerns you, it may be wiser to instead prefer only granting these capabilities to a dedicated Fail2Ban container ([example][lsio:f2b-image]).
 
 !!! bug "Running Fail2Ban on Older Kernels"
 
-    DMS configures F2B to use [NFTables][network::nftables], not [IPTables (legacy)][network::iptables-legacy].
+    DMS configures F2B to use NFTables, not IPTables (legacy). We have observed that older systems, for example NAS systems, do not support the modern NFTables rules. You will need to configure F2B to use legacy IPTables again, for example with the [``fail2ban-jail.cf``][github-file-f2bjail], see the [section on configuration further down below](#custom-files).
-
-    We have observed that older systems (for example NAS systems), do not support the modern NFTables rules. You will need to configure F2B to use legacy IPTables again, for example with the [`fail2ban-jail.cf`][github-file-f2bjail], see the [section on configuration further down below](#custom-files).
-
-[security::cap-net-admin]: https://0xn3va.gitbook.io/cheat-sheets/container/escaping/excessive-capabilities#cap_net_admin
-[security::cap-net-raw]: https://0xn3va.gitbook.io/cheat-sheets/container/escaping/excessive-capabilities#cap_net_raw
-[lsio:f2b-image]: https://docs.linuxserver.io/images/docker-fail2ban
-[network::nftables]: https://en.wikipedia.org/wiki/Nftables
-[network::iptables-legacy]: https://developers.redhat.com/blog/2020/08/18/iptables-the-two-variants-and-their-relationship-with-nftables#two_variants_of_the_iptables_command
 
 ### DMS Defaults
 

docs/content/config/security/rspamd.md

@@ -6,60 +6,34 @@ title: 'Security | Rspamd'
 
 Rspamd is a ["fast, free and open-source spam filtering system"][rspamd-web]. DMS integrates Rspamd like any other service. We provide a basic but easy to maintain setup of Rspamd.
 
-If you want to take a look at the default configuration files for Rspamd that DMS adds, navigate to [`target/rspamd/` inside the repository][dms-repo::default-rspamd-configuration]. Please consult the [section "The Default Configuration"](#the-default-configuration) section down below for a written overview.
+If you want to take a look at the default configuration files for Rspamd that DMS packs, navigate to [`target/rspamd/` inside the repository][dms-repo::default-rspamd-configuration]. Please consult the [section "The Default Configuration"](#the-default-configuration) section down below for a written overview.
 
-### Enable Rspamd
+## Related Environment Variables
 
-Rspamd is presently opt-in for DMS, but intended to become the default anti-spam service in a future release.
+The following environment variables are related to Rspamd:
 
-DMS offers two anti-spam solutions:
+1. [`ENABLE_RSPAMD`](../environment.md#enable_rspamd)
+2. [`ENABLE_RSPAMD_REDIS`](../environment.md#enable_rspamd_redis)
+3. [`RSPAMD_CHECK_AUTHENTICATED`](../environment.md#rspamd_check_authenticated)
+4. [`RSPAMD_GREYLISTING`](../environment.md#rspamd_greylisting)
+5. [`RSPAMD_HFILTER`](../environment.md#rspamd_hfilter)
+6. [`RSPAMD_HFILTER_HOSTNAME_UNKNOWN_SCORE`](../environment.md#rspamd_hfilter_hostname_unknown_score)
+7. [`RSPAMD_LEARN`](../environment.md#rspamd_learn)
+8. [`SPAM_SUBJECT`](../environment.md#spam_subject)
+9. [`MOVE_SPAM_TO_JUNK`][docs::spam-to-junk]
+10. [`MARK_SPAM_AS_READ`](../environment.md#mark_spam_as_read)
 
-- Legacy (_Amavis, SpamAssassin, OpenDKIM, OpenDMARC_)
+With these variables, you can enable Rspamd itself, and you can enable / disable certain features related to Rspamd.
-- Rspamd (_Provides equivalent features of software from the legacy solution_)
 
-While you could configure Rspamd to only replace some of the legacy services, it is advised to only use Rspamd with the legacy services disabled.
+## The Default Configuration
 
-!!! example "Switch to Rspamd"
+### Other Anti-Spam-Services
 
-    To use Rspamd add the following ENV config changes:
+DMS packs other anti-spam services, like SpamAssassin or Amavis, next to Rspamd. There exist services, like ClamAV (`ENABLE_CLAMAV`), that Rspamd can utilize to improve the scanning. Except for ClamAV, we recommend disabling **all other** anti-spam services when using Rspamd. The [basic configuration shown below](#a-very-basic-configuration) provides a good starting point.
-    
-    ```env
-    ENABLE_RSPAMD=1
-    
-    # Rspamd replaces the functionality of all these anti-spam services, disable them:
-    ENABLE_OPENDKIM=0
-    ENABLE_OPENDMARC=0
-    ENABLE_POLICYD_SPF=0
-    ENABLE_AMAVIS=0
-    ENABLE_SPAMASSASSIN=0
-    # Greylisting is opt-in, if you had enabled Postgrey switch to the Rspamd equivalent:
-    ENABLE_POSTGREY=0
-    RSPAMD_GREYLISTING=1
-    
-    # Optional: Add anti-virus support with ClamAV (compatible with Rspamd):
-    ENABLE_CLAMAV=1
-    ```
-
-!!! info "Relevant Environment Variables"
-
-    The following environment variables are related to Rspamd:
-    
-    1. [`ENABLE_RSPAMD`](../environment.md#enable_rspamd)
-    2. [`ENABLE_RSPAMD_REDIS`](../environment.md#enable_rspamd_redis)
-    3. [`RSPAMD_CHECK_AUTHENTICATED`](../environment.md#rspamd_check_authenticated)
-    4. [`RSPAMD_GREYLISTING`](../environment.md#rspamd_greylisting)
-    5. [`RSPAMD_HFILTER`](../environment.md#rspamd_hfilter)
-    6. [`RSPAMD_HFILTER_HOSTNAME_UNKNOWN_SCORE`](../environment.md#rspamd_hfilter_hostname_unknown_score)
-    7. [`RSPAMD_LEARN`](../environment.md#rspamd_learn)
-    8. [`SPAM_SUBJECT`](../environment.md#spam_subject)
-    9. [`MOVE_SPAM_TO_JUNK`][docs::spam-to-junk]
-    10. [`MARK_SPAM_AS_READ`](../environment.md#mark_spam_as_read)
-
-## Overview of Rspamd support
 
 ### Mode of Operation
 
-!!! note "Attention"
+!!! tip "Attention"
 
     Read this section carefully if you want to understand how Rspamd is integrated into DMS and how it works (on a surface level).
 
@@ -105,16 +79,13 @@ DMS does not set a default password for the controller worker. You may want to d
 When Rspamd is enabled, we implicitly also start an instance of Redis in the container:
 
 - Redis is configured to persist its data via RDB snapshots to disk in the directory `/var/lib/redis` (_or the [`/var/mail-state/`][docs::dms-volumes-state] volume when present_).
-- With the volume mount, the snapshot will restore the Redis data across container updates, and provide a way to keep a backup.
+- With the volume mount, the snapshot will restore the Redis data across container restarts, and provide a way to keep backup.
-- Without a volume mount a containers internal state will persist across restarts until the container is recreated due to changes like ENV or upgrading the image for the container.
 
 Redis uses `/etc/redis/redis.conf` for configuration:
 
 - We adjust this file when enabling the internal Redis service.
 - If you have an external instance of Redis to use, the internal Redis service can be opt-out via setting the ENV [`ENABLE_RSPAMD_REDIS=0`][docs::env::enable-redis] (_link also details required changes to the DMS Rspamd config_).
 
-If you are interested in using Valkey instead of Redis, please refer to [this guidance][gh-dms::guide::valkey].
-
 ### Web Interface
 
 Rspamd provides a [web interface][rspamd-docs::web-ui], which contains statistics and data Rspamd collects. The interface is enabled by default and reachable on port 11334.
@@ -125,7 +96,7 @@ To use the web interface you will need to configure a password, [otherwise you w
 
 ??? example "Set a custom password"
 
-    Add this line to [your Rspamd `custom-commands.conf` config](#with-the-help-of-a-custom-file) which sets the `password` option of the _controller worker_:
+    Add this line to [your rspamd `custom-commands.conf` config](#with-the-help-of-a-custom-file) which sets the `password` option of the _controller worker_:
 
     ```
     set-option-for-controller password "your hashed password here"
@@ -137,13 +108,9 @@ To use the web interface you will need to configure a password, [otherwise you w
     docker exec -it <CONTAINER_NAME> rspamadm pw
     ```
 
-    ---
-
-    **Related:** A minimal Rspamd `compose.yaml` [example with a reverse-proxy for web access][gh-dms::guide::rspamd-web].
-
 ### DNS
 
-DMS does not supply custom values for DNS servers (to Rspamd). If you need to use custom DNS servers, which could be required when using [DNS-based deny/allowlists](#rbls-real-time-blacklists-dnsbls-dns-based-blacklists), you need to adjust [`options.inc`][rspamd-docs::config::global] yourself. Make sure to also read our [FAQ page on DNS servers][docs::faq::dns-servers].
+DMS does not supply custom values for DNS servers (to Rspamd). If you need to use custom DNS servers, which could be required when using [DNS-based deny/allowlists](#rbls-real-time-blacklists-dnsbls-dns-based-blacklists), you need to adjust [`options.inc`][rspamd-docs::basic-options] yourself. Make sure to also read our [FAQ page on DNS servers][docs::faq::dns-servers].
 
 !!! warning
 
@@ -175,7 +142,7 @@ You can choose to enable ClamAV, and Rspamd will then use it to check for viruse
 
 #### RBLs (Real-time Blacklists) / DNSBLs (DNS-based Blacklists)
 
-The [RBL module][rspamd-docs::modules::rbl] is enabled by default. As a consequence, Rspamd will perform DNS lookups to various blacklists. Whether an RBL or a DNSBL is queried depends on where the domain name was obtained: RBL servers are queried with IP addresses extracted from message headers, DNSBL server are queried with domains and IP addresses extracted from the message body ([source][www::rbl-vs-dnsbl]).
+The [RBL module][rspamd-docs::modules::rbl] is enabled by default. As a consequence, Rspamd will perform DNS lookups to various blacklists. Whether an RBL or a DNSBL is queried depends on where the domain name was obtained: RBL servers are queried with IP addresses extracted from message headers, DNSBL server are queried with domains and IP addresses extracted from the message body \[[source][www::rbl-vs-dnsbl]\].
 
 !!! danger "Rspamd and DNS Block Lists"
 
@@ -185,141 +152,124 @@ The [RBL module][rspamd-docs::modules::rbl] is enabled by default. As a conseque
 
 ## Providing Custom Settings & Overriding Settings
 
-!!! info "Rspamd config overriding precedence"
+DMS brings sane default settings for Rspamd. They are located at `/etc/rspamd/local.d/` inside the container (or `target/rspamd/local.d/` in the repository).
 
-    Rspamd has a layered approach for configuration with [`local.d` and `override.d` config directories][rspamd-docs::config-directories].
+### Manually
 
-    - DMS [extends the Rspamd default configs via `/etc/rspamd/local.d/`][dms-repo::default-rspamd-configuration].
+!!! question "What is [`docker-data/dms/config/`][docs::dms-volumes-config]?"
-    - User config changes should be handled separately as overrides via the [DMS Config Volume][docs::dms-volumes-config] (`docker-data/dms/config/`) with either:
-        - `./rspamd/override.d/` - Config files placed here are copied to `/etc/rspamd/override.d/` during container startup.
-        - [`./rspamd/custom-commands.conf`](#with-the-help-of-a-custom-file) - Applied after copying any provided configs from `rspamd/override.d/` (DMS Config volume) to `/etc/rspamd/override.d/`.
 
-!!! abstract "Reference docs for Rspamd config"
+If you want to overwrite the default settings or provide your settings, you can place files at `docker-data/dms/config/rspamd/override.d/`. Files from this directory are copied to `/etc/rspamd/override.d/` during startup. These files [forcibly override][rspamd-docs::override-dir] Rspamd and DMS default settings.
 
-    - [Config Overview][rspamd-docs::config::overview], [Quickstart guide][rspamd-docs::config::quickstart], and [Config Syntax (UCL)][rspamd-docs::config::ucl-syntax]
+!!! question "What is the [`local.d` directory and how does it compare to `override.d`][rspamd-docs::config-directories]?"
-    - Global Options ([`options.inc`][rspamd-docs::config::global])
-    - [Workers][rspamd-docs::config::workers] ([`worker-controller.inc`][rspamd-docs::config::worker-controller], [`worker-proxy.inc`][rspamd-docs::config::worker-proxy])
-    - [Modules][rspamd-docs::modules] (_view each module page for their specific config options_)
 
-!!! tip "View rendered config"
+!!! warning "Clashing Overrides"
 
-    `rspamadm configdump` will output the full rspamd configuration that is used should you need it for troubleshooting / inspection.
+    Note that when also [using the `custom-commands.conf` file](#with-the-help-of-a-custom-file), files in `override.d` may be overwritten in case you adjust them manually and with the help of the file.
 
-    - You can also see which modules are enabled / disabled via `rspamadm configdump --modules-state`
+### With the Help of a Custom File
-    - Specific config sections like `dkim` or `worker` can also be used to filter the output to just those sections: `rspamadm configdump dkim worker`
-    - Use `--show-help` to include inline documentation for many settings.
 
-### Using `custom-commands.conf` { #with-the-help-of-a-custom-file }
+DMS provides the ability to do simple adjustments to Rspamd modules with the help of a single file. Just place a file called `custom-commands.conf` into `docker-data/dms/config/rspamd/`. If this file is present, DMS will evaluate it. The structure is simple, as each line in the file looks like this:
 
-For convenience DMS provides a single config file that will directly create or modify multiple configs at `/etc/rspamd/override.d/`. This is handled as the final rspamd configuration step during container startup.
+```txt
+COMMAND ARGUMENT1 ARGUMENT2 ARGUMENT3
+```
 
-DMS will apply this config when you provide `rspamd/custom-commands.conf` in your DMS Config volume. Configure it with directive lines as documented below.
+where `COMMAND` can be:
 
-!!! note "Only use this feature for `option = value` changes"
+1. `disable-module`: disables the module with name `ARGUMENT1`
+2. `enable-module`: explicitly enables the module with name `ARGUMENT1`
+3. `set-option-for-module`: sets the value for option `ARGUMENT2` to `ARGUMENT3` inside module `ARGUMENT1`
+4. `set-option-for-controller`: set the value of option `ARGUMENT1` to `ARGUMENT2` for the controller worker
+5. `set-option-for-proxy`: set the value of option `ARGUMENT1` to `ARGUMENT2` for the proxy worker
+6. `set-common-option`: set the option `ARGUMENT1` that [defines basic Rspamd behavior][rspamd-docs::basic-options] to value `ARGUMENT2`
+7. `add-line`: this will add the complete line after `ARGUMENT1` (with all characters) to the file `/etc/rspamd/override.d/<ARGUMENT1>`
 
-    `custom-commands.conf` is only suitable for adding or replacing simple `option = value` settings for configs at `/etc/rspamd/override.d/`.
+!!! example "An Example Is [Shown Down Below](#adjusting-and-extending-the-very-basic-configuration)"
 
-    - New settings are appended to the associated config file.
+!!! note "File Names & Extensions"
-    - When replacing an existing setting in an override config, that setting may be any matching line (_allowing for nested scopes, instead of only top-level keys_).
 
-    Any changes involving more advanced [UCL config syntax][rspamd-docs::config::ucl-syntax] should instead add UCL config files directly to `rspamd/override.d/` (_in the DMS Config volume_).
+    For command 1 - 3, we append the `.conf` suffix to the module name to get the correct file name automatically. For commands 4 - 6, the file name is fixed (you don't even need to provide it). For command 7, you will need to provide the whole file name (including the suffix) yourself!
 
-!!! info "`custom-commands.conf` syntax"
+You can also have comments (the line starts with `#`) and blank lines in `custom-commands.conf` - they are properly handled and not evaluated.
 
-    There are 7 directives available to manage custom Rspamd configurations. Add these directive lines into `custom-commands.conf`, they will be processed sequentially.
+!!! tip "Adjusting Modules This Way"
 
-    **Directives:**
+    These simple commands are meant to give users the ability to _easily_ alter modules and their options. As a consequence, they are not powerful enough to enable multi-line adjustments. If you need to do something more complex, we advise to do that [manually](#manually)!
 
-    ```txt
+## Examples & Advanced Configuration
-    # For /etc/rspamd/override.d/{options.inc,worker-controller.inc,worker-proxy}.inc
-    set-common-option         <OPTION NAME> <OPTION VALUE>
-    set-option-for-controller <OPTION NAME> <OPTION VALUE>
-    set-option-for-proxy      <OPTION NAME> <OPTION VALUE>
 
-    # For /etc/rspamd/override.d/<MODULE NAME>.conf
+### A Very Basic Configuration
-    enable-module         <MODULE NAME>
-    disable-module        <MODULE NAME>
-    set-option-for-module <MODULE NAME> <OPTION NAME> <OPTION VALUE>
 
-    # For /etc/rspamd/override.d/<FILENAME>
+Do you want to start using Rspamd? Rspamd is disabled by default, so you need to set the following environment variables:
-    add-line <FILENAME> <CONTENT>
-    ```
 
-    **Syntax:**
+```env
+ENABLE_RSPAMD=1
+# ClamAV is compatible with Rspamd. Optionally enable it for anti-virus support:
+ENABLE_CLAMAV=1
 
-    - Blank lines are ok.
+# Rspamd replaces the functionality of all these anti-spam services, disable them:
-    - `#` at the start of a line represents a comment for adding notes.
+ENABLE_OPENDKIM=0
-    - `<OPTION VALUE>` and `<CONTENT>` will contain the remaining content of their line, any preceding inputs are delimited by white-space.
+ENABLE_OPENDMARC=0
+ENABLE_POLICYD_SPF=0
+ENABLE_AMAVIS=0
+ENABLE_SPAMASSASSIN=0
 
-    ---
+# Provided you've set `RSPAMD_GREYLISTING=1`, also disable Postgrey:
+ENABLE_POSTGREY=0
+```
 
-    ??? note "`<MODULE NAME>` can also target non-module configs"
+This will enable Rspamd and disable services you don't need when using Rspamd.
 
-        An example is the `statistics` module, which has config to import a separate file (`classifier-bayes.conf`) for easier overrides to this section of the module config.
+### Adjusting and Extending The Very Basic Configuration
 
-??? example
+Rspamd is running, but you want or need to adjust it? First, create a file named `custom-commands.conf` under `docker-data/dms/config/rspamd` (which translates to `/tmp/docker-mailserver/rspamd/` inside the container). Then add your changes:
 
-    ```conf title="rspamd/custom-commands.conf"
+1. Say you want to be able to easily look at the frontend Rspamd provides on port 11334 (default) without the need to enter a password (maybe because you already provide authorization and authentication). You will have to adjust the controller worker: `set-option-for-controller secure_ip "0.0.0.0/0"`.
-    # If you're confident you've properly secured access to the rspamd web service/API (Default port: 11334)
+2. Do you additionally want to enable the auto-spam-learning for the Bayes module? No problem: `set-option-for-module classifier-bayes autolearn true`.
-    # with your own auth layer (eg: reverse-proxy) you can bypass rspamd requiring credentials:
+3. But the chartable module gets on your nerves? Easy: `disable-module chartable`.
-    # https://rspamd.com/doc/workers/controller.html#controller-configuration
+
+??? example "What Does the Result Look Like?"
+    Here is what the file looks like in the end:
+
+    ```bash
+    # See 1.
+    # ATTENTION: this disables authentication on the website - make sure you know what you're doing!
     set-option-for-controller secure_ip "0.0.0.0/0"
 
-    # Some settings aren't documented well, you may find them in snippets or Rspamds default config files:
+    # See 2.
-    # https://rspamd.com/doc/tutorials/quickstart.html#using-of-milter-protocol-for-rspamd--16
-    # /etc/rspamd/worker-proxy.inc
-    set-option-for-proxy reject_message "Rejected - Detected as spam"
-
-    # Equivalent to the previous example, but `add-line` is more verbose:
-    add-line worker-proxy.inc reject_message = "Rejected - Detected as spam"
-
-    # Enable Bayes auto-learning feature to classify spam based on Rspamd action/score results:
-    # NOTE: The statistics module imports a separate file for classifier-bayes config
-    # https://rspamd.com/doc/configuration/statistic.html#autolearning
     set-option-for-module classifier-bayes autolearn true
 
-    # Disable the `chartable` module:
+    # See 3.
-    # https://rspamd.com/doc/modules/chartable.html
     disable-module chartable
     ```
 
-## Advanced Configuration
-
 ### DKIM Signing
 
 There is a dedicated [section for setting up DKIM with Rspamd in our documentation][docs::dkim-with-rspamd].
 
 ### ARC (Authenticated Received Chain)
 
-[ARC][wikipedia::arc] support in DMS is opt-in via config file. [Enable the ARC Rspamd module][rspamd-docs::arc] by creating a config file at `docker-data/dms/config/rspamd/override.d/arc.conf`.
+ARC is not set up by default, but you can easily enable it by adding a file called `arc.conf` to `docker-data/dms/config/rspamd/override.d/`. ARC can use DKIM keys that you should have already created. The configuration file could then contain the following:
 
-!!! example
+```conf
+sign_local = true;
+sign_authenticated = true;
 
-    For each mail domain you have DMS manage, add the equivalent `example.com` sub-section to `domain` and adjust the `path` + `selector` fields as necessary.
+domain {
-
+    <DOMAIN NAME> {
-    ```conf title="rspamd/override.d/arc.conf"
+        # Change the path here to your actual private key
-    sign_local = true;
+        path = "/tmp/docker-mailserver/rspamd/dkim/rsa-2048-mail-<DOMAIN NAME>.private.txt";
-    sign_authenticated = true;
+        # Changhe the selected if you chose a non-default one
-
+        selector = "mail";
-    domain {
-        example.com {
-            path = "/tmp/docker-mailserver/rspamd/dkim/rsa-2048-mail-example.private.txt";
-            selector = "mail";
-        }
     }
-    ```
+}
-
+```
-!!! tip "Using a common keypair"
-
-    As with DKIM, the keypair can be shared across your configured domains.
-
-    Your ARC config can share the same DKIM private key + selector (_with associated DNS record for the public key_).
 
 ### _Abusix_ Integration
 
 This subsection provides information about the integration of [Abusix][abusix-web], "a set of blocklists that work as an additional email security layer for your existing mail environment". The setup is straight-forward and well documented:
 
-1. [Create an account][abusix-web::register]
+1. [Create an account](https://app.abusix.com/)
 2. Retrieve your API key
 3. Navigate to the ["Getting Started" documentation for Rspamd][abusix-docs::rspamd-integration] and follow the steps described there
 4. Make sure to change `<APIKEY>` to your private API key
@@ -335,28 +285,17 @@ While _Abusix_ can be integrated into Postfix, Postscreen and a multitude of oth
 [rspamd-docs::web-ui::password]: https://www.rspamd.com/doc/tutorials/quickstart.html#setting-the-controller-password
 [rspamd-docs::modules]: https://rspamd.com/doc/modules/
 [rspamd-docs::modules::rbl]: https://rspamd.com/doc/modules/rbl.html
+[rspamd-docs::override-dir]: https://www.rspamd.com/doc/faq.html#what-are-the-locald-and-overrided-directories
 [rspamd-docs::config-directories]: https://rspamd.com/doc/faq.html#what-are-the-locald-and-overrided-directories
-[rspamd-docs::config::ucl-syntax]: https://rspamd.com/doc/configuration/ucl.html
+[rspamd-docs::basic-options]: https://rspamd.com/doc/configuration/options.html
-[rspamd-docs::config::overview]: https://rspamd.com/doc/configuration/index.html
-[rspamd-docs::config::quickstart]: https://rspamd.com/doc/tutorials/quickstart.html#configuring-rspamd
-[rspamd-docs::config::global]: https://rspamd.com/doc/configuration/options.html
-[rspamd-docs::config::workers]: https://rspamd.com/doc/workers/
-[rspamd-docs::config::worker-controller]: https://rspamd.com/doc/workers/controller.html
-[rspamd-docs::config::worker-proxy]: https://rspamd.com/doc/workers/rspamd_proxy.html
 
-[wikipedia::arc]: https://en.wikipedia.org/wiki/Authenticated_Received_Chain
+[www::rbl-vs-dnsbl]: https://forum.eset.com/topic/25277-dnsbl-vs-rbl-mail-security/?do=findComment&comment=119818
-[rspamd-docs::arc]: https://rspamd.com/doc/modules/arc.html
-
-[www::rbl-vs-dnsbl]: https://forum.eset.com/topic/25277-dnsbl-vs-rbl-mail-security/#comment-119818
 [abusix-web]: https://abusix.com/
-[abusix-web::register]: https://app.abusix.com/
 [abusix-docs::rspamd-integration]: https://abusix.com/docs/rspamd/
 [spamhaus::faq::dnsbl-usage]: https://www.spamhaus.org/faq/section/DNSBL%20Usage#365
 
-[dms-repo::rspamd-actions-config]: https://github.com/docker-mailserver/docker-mailserver/tree/v15.0.0/target/rspamd/local.d/actions.conf
+[dms-repo::rspamd-actions-config]: https://github.com/docker-mailserver/docker-mailserver/blob/v14.0.0/target/rspamd/local.d/actions.conf
-[dms-repo::default-rspamd-configuration]: https://github.com/docker-mailserver/docker-mailserver/tree/v15.0.0/target/rspamd
+[dms-repo::default-rspamd-configuration]: https://github.com/docker-mailserver/docker-mailserver/tree/v14.0.0/target/rspamd
-[gh-dms::guide::valkey]: https://github.com/docker-mailserver/docker-mailserver/issues/4001#issuecomment-2652596692
-[gh-dms::guide::rspamd-web]: https://github.com/orgs/docker-mailserver/discussions/4269#discussioncomment-11329588
 
 [docs::env::enable-redis]: ../environment.md#enable_rspamd_redis
 [docs::spam-to-junk]: ../environment.md#move_spam_to_junk

docs/content/contributing/general.md

@@ -15,20 +15,13 @@ When refactoring, writing or altering scripts or other files, adhere to these ru
 
 Make sure to select `edge` in the dropdown menu at the top. Navigate to the page you would like to edit and click the edit button in the top right. This allows you to make changes and create a pull-request.
 
-Alternatively you can make the changes locally. For that you'll need to have Docker installed and run:
+Alternatively you can make the changes locally. For that you'll need to have Docker installed. Navigate into the `docs/` directory. Then run:
 
 ```sh
-# From the root directory of the git clone:
+docker run --rm -it -p 8000:8000 -v "${PWD}:/docs" squidfunk/mkdocs-material
-docker run --rm -it -p 8000:8000 -v "./docs:/docs" squidfunk/mkdocs-material
 ```
 
 This serves the documentation on your local machine on port `8000`. Each change will be hot-reloaded onto the page you view, just edit, save and look at the result.
 
-!!! note
-
-    The container logs will inform you of invalid links detected, but a [few are false-positives][gh-dms::mkdocs-link-error-false-positives] due to our usage of linking to specific [content tabs][mkdocs::content-tabs].
-
 [get-docker]: https://docs.docker.com/get-docker/
 [docs-bats-parallel]: https://bats-core.readthedocs.io/en/v1.8.2/usage.html#parallel-execution
-[gh-dms::mkdocs-link-error-false-positives]: https://github.com/docker-mailserver/docker-mailserver/pull/4366
-[mkdocs::content-tabs]: https://squidfunk.github.io/mkdocs-material/reference/content-tabs/#anchor-links

docs/content/examples/tutorials/dovecot-solr.md

@@ -44,21 +44,19 @@ As the official DMS image does not provide `dovecot-solr`, you'll need to includ
           mailserver:
             hostname: mail.example.com
             # The `image` setting now represents the tag for the local build configured below:
-            image: local/dms:${DMS_TAG?Must set DMS image tag}
+            image: local/dms:14.0
             # Local build (no need to try pull `image` remotely):
             pull_policy: build
             # Add this `build` section to your real `compose.yaml` for your DMS service:
             build:
               dockerfile_inline: |
-                FROM docker.io/mailserver/docker-mailserver:${DMS_TAG?Must set DMS image tag}
+                FROM docker.io/mailserver/docker-mailserver:14.0
                 RUN apt-get update && apt-get install dovecot-solr
         ```
     
-        This approach only needs to install the package once with the image build itself which minimizes the delay of container startup.
+        - Just run `docker compose up` and it will pull DMS and build your custom image to run a container.
-
+        - Updating to a new DMS release is straight-forward, just adjust the version tag as you normally would. If you make future changes that don't apply, you may need to force a rebuild.
-        - Just run `DMS_TAG='14.0' docker compose up` and it will pull the DMS image, then build your custom DMS image to run a new container instance.
+        - This approach only needs to install the package once with the image build itself. This minimizes delay of container startup.
-        - Updating to a new DMS release is straight-forward, just adjust the `DMS_TAG` ENV value or change the image tag directly in `compose.yaml` as you normally would to upgrade an image.
-        - If you make future changes to the `dockerfile_inline` that don't seem to be applied, you may need to force a rebuild with `DMS_TAG='14.0' docker compose up --build`.
 
 !!! note "Why doesn't DMS include `dovecot-solr`?"
 
@@ -146,25 +144,6 @@ docker compose exec mailserver doveadm fts rescan -A
 
     Usually within 15 minutes or so, you should be able to search your mail using the Dovecot FTS feature! :tada:
 
-### Compatibility
-
-Since Solr 9.8.0 was released (Jan 2025), a breaking change [deprecates support for `<lib>` directives][solr::9.8::lib-directive] which is presently used by the Dovecot supplied Solr config (`solr-config-9.xml`) to automatically load additional jars required.
-
-To enable support for `<lib>` directives, add the following ENV to your `solr` container:
-
-```yaml
-services:
-  solr:
-    environment:
-      SOLR_CONFIG_LIB_ENABLED: true
-```
-
-!!! warning "Solr 10"
-
-    From the Solr 10 release onwards, this opt-in ENV will no longer be available.
-
-    If Dovecot has not updated their example Solr config ([upstream PR][dovecot::pr::solr-config-lib]), you will need to manually modify the Solr XML config to remove the `<lib>` directives and replace the suggested ENV `SOLR_CONFIG_LIB_ENABLED=true` with `SOLR_MODULES=analysis-extras`.
-
 [docs::user-patches]: ../../config/advanced/override-defaults/user-patches.md
 [docs::dovecot::full-text-search]: ../../config/advanced/full-text-search.md
 [gh-dms::feature-request::dovecot-solr-package]: https://github.com/docker-mailserver/docker-mailserver/issues/4052
@@ -173,6 +152,3 @@ services:
 [dockerfile-solr-uidgid]: https://github.com/apache/solr-docker/blob/9cd850b72309de05169544395c83a85b329d6b86/9.6/Dockerfile#L89-L92
 [github-solr]: https://github.com/apache/solr
 [github-dovecot::core-docs]: https://github.com/dovecot/core/tree/main/doc
-
-[solr::9.8::lib-directive]: https://issues.apache.org/jira/browse/SOLR-16781
-[dovecot::pr::solr-config-lib]: https://github.com/dovecot/core/pull/238

docs/content/examples/use-cases/bind-smtp-network-interface.md

@@ -6,20 +6,27 @@ hide:
 
 !!! warning "Advice not extensively tested"
 
-    This configuration advice is a community contribution which has only been verified as a solution when using `network: host`, where you have direct access to the host interfaces.
+    This configuration advice is a community contribution.
 
-    It may be applicable in other network modes if the container has control of the outbound IPs to bind to. This is not the case with bridge networks that typically bind to a private range network for containers which are bridged to a public interface via Docker.
+If your Docker host has multiple public IPv4 or IPv6 IP addresses, it may be beneficial to bind outgoing SMTP connections to a specific IP.
 
-If your Docker host is running multiple IPv4 and IPv6 IP-addresses, it may be beneficial to bind outgoing SMTP connections to specific IP-address / interface.
+When the IP address for DMS is inconsistent, it may fail to pass common security checks when interacting with other mail servers:
 
-- When a mail is sent outbound from DMS, it greets the MTA it is connecting to with a EHLO (DMS FQDN) which might be verified against the IP resolved, and that a `PTR` record for that IP resolves an address back to the same IP.
+- When a mail is sent outbound from DMS, it connects to the external MTA and greets it with an EHLO (the DMS FQDN) which the MTA might verify that the EHLO address resolves to the same IP that DMS is connecting to the MTA from, and that a `PTR` record for that same IP resolves back to that same IP (_the `PTR` record address may not need to match the DMS FQDN_).
-- A similar check with SPF can be against the envelope-sender address which may verify a DNS record like MX / A is valid (_or a similar restriction check from an MTA like [Postfix has with `reject_unknown_sender`][gh-pr::3465::comment-restrictions]_).
+- SPF is common requirement which verifies the SPF `TXT` DNS record for the _envelope sender address_ is valid when it references any DNS values (`MX`, `A`, `AAAA`).
-- If the IP address is inconsistent for those connections from DMS, these DNS checks are likely to fail.
+- There may be other security checks handled by an MTA, like [Postfix supports with `reject_unknown_sender`][gh-pr::3465::comment-restrictions].
+
+!!! note "IP addresses for documentation"
+
+    IP addresses shown in the below examples (`198.51.100.42` + `2001:DB8::42`) are placeholders, they are IP addresses reserved for documentation by IANA (_[RFC-5737 (IPv4)][rfc-5737] and [RFC-3849 (IPv6)][rfc-3849]_). Replace them with the IP addresses you want DMS to send mail through.
+
+## Directly binding
+
+This approach is for when the DMS container uses a [`network_mode`][docker-docs::compose-config::network-mode] / [`networks`][docker-docs::compose-config::networks] where the host IP is available to bind internally, such as [host][docker-docs::network-driver::host] and [macvlan][docker-docs::network-driver::macvlan].
 
 This can be configured by [overriding the default Postfix configurations][docs::overrides-postfix] DMS provides. Create `postfix-master.cf` and `postfix-main.cf` files for your config volume (`docker-data/dms/config`).
 
-In `postfix-main.cf` you'll have to set the [`smtp_bind_address`][postfix-docs::smtp-bind-address-ipv4] and [`smtp_bind_address6`][postfix-docs::smtp-bind-address-ipv6]
+In `postfix-main.cf` you'll have to set the [`smtp_bind_address`][postfix-docs::smtp-bind-address-ipv4] and [`smtp_bind_address6`][postfix-docs::smtp-bind-address-ipv6] to the respective IP-address on the server you want to use.
-to the respective IP-address on the server you want to use.
 
 !!! example
 
@@ -32,7 +39,9 @@ to the respective IP-address on the server you want to use.
 
         !!! bug "Inheriting the bind from `main.cf` can misconfigure services"
 
-            One problem when setting `smtp_bind_address` in `main.cf` is that it will be inherited by any services in `master.cf` that extend the `smtp` transport. One of these is `smtp-amavis`, which is explicitly configured to listen / connect via loopback (localhost / `127.0.0.1`).
+            One problem when setting `smtp_bind_address` in `main.cf` is that it will be inherited by any services in `master.cf` that extend the `smtp` transport.
+
+            One of these is `smtp-amavis`, which is explicitly configured to listen / connect via loopback (`localhost` / `127.0.0.1`).
 
             A `postfix-master.cf` override can workaround that issue by ensuring `smtp-amavis` binds to the expected internal IP:
 
@@ -50,42 +59,45 @@ to the respective IP-address on the server you want to use.
         smtp/inet/smtp_bind_address6 = 2001:DB8::42
         ```
 
-        If that avoids the concern with `smtp-amavis`, you may still need to additionally override for the [`relay` transport][gh-src::postfix-master-cf::relay-transport] as well if you have configured DMS to relay mail.
+        If that avoids the concern with `smtp-amavis`, you may still need to additionally override for the [`relay` transport][gh-src::postfix-master-cf::relay-transport] as well if you have configured DMS to send mail through a relay service.
         
-    === "Bridged Networks"
+## Bridged Networks
     
-        When your DMS container is using a bridge network, you'll instead need to restrict which IP address inbound and outbound traffic is routed through via the bridged interface.
+When your DMS container is using a bridge network, you'll instead need to restrict which IP address inbound and outbound traffic is routed through via the bridged interface.
 
-        For **inbound** traffic, you may configure this at whatever scope is most appropriate for you:
+!!! info
 
-        - **Daemon:** Change the [default bind address][inbound-ip::docker-docs::daemon] configured in `/etc/docker/daemon.json` (default `0.0.0.0`)
+    For **inbound** traffic, you may configure this at whatever scope is most appropriate for you:
-        - **Network:** Assign the [`host_binding_ipv4` bridge driver option][inbound-ip::docker-docs::network] as shown in the below `compose.yaml` snippet.
-        - **Container:** Provide an explicit host IP address when [publishing a port][inbound-ip::docker-docs::container].
     
-        For **outbound** traffic, the bridge network will use the default route. You can change this by either:
+    - **Daemon:** Change the [default bind address][inbound-ip::docker-docs::daemon] configured in `/etc/docker/daemon.json` (default `0.0.0.0`).
+    - **Network:** Assign the [`host_binding_ipv4` bridge driver option][inbound-ip::docker-docs::network] as shown in the below `compose.yaml` snippet.
+    - **Container:** Provide an explicit host IP address when [publishing a port][inbound-ip::docker-docs::container].
     
-        - [Manually routing networks][outbound-ip::route-manually] on the host.
+    For **outbound** traffic, the bridge network will use the default route. You can change this by either:
-        - Use the [`host_ipv4` driver option][outbind-ip::host-ipv4] for Docker networks to force the SNAT (source IP) that the bridged network will route outbound traffic through.
-            - This IP address must belong to a network interface to be routed through it.
-            - IPv6 support via `host_ipv6` [requires at least Docker v25][outbind-ip::host-ipv6].
     
-        ---
+    - [Manually routing networks][outbound-ip::route-manually] on the host.
+    - Use the [`host_ipv4` driver option][outbind-ip::host-ipv4] for Docker networks to force the SNAT (source IP) that the bridged network will route outbound traffic through.
+        - This IP address must belong to a network interface to be routed through it.
+        - IPv6 support via `host_ipv6` [requires at least Docker v25][outbind-ip::host-ipv6].
 
-        Here is a `compose.yaml` snippet that applies the inbound + outbound settings to the default bridge network Docker Compose creates (_if it already exists, you will need to ensure it's re-created to apply the updated settings_):
+!!! example
 
-        ```yaml title="compose.yaml"
+    Here is a `compose.yaml` snippet that applies the inbound + outbound settings to the default bridge network Docker Compose creates (_if it already exists, you will need to ensure it's re-created to apply the updated settings_):
-        networks:
-          default:
-            driver_opts:
-              # Inbound IP (sets the host IP that published ports receive traffic from):
-              com.docker.network.bridge.host_binding_ipv4: 198.51.100.42
-              # Outbound IP (sets the host IP that external hosts will receive connections from):
-              com.docker.network.host_ipv4: 198.51.100.42
-        ```
     
-!!! note "IP addresses for documentation"
+    ```yaml title="compose.yaml"
+    networks:
+      default:
+        driver_opts:
+          # Inbound IP (sets the host IP that published ports receive traffic from):
+          com.docker.network.bridge.host_binding_ipv4: 198.51.100.42
+          # Outbound IP (sets the host IP that external hosts will receive connections from):
+          com.docker.network.host_ipv4: 198.51.100.42
+    ```
 
-    IP addresses shown in above examples (`198.51.100.42` + `2001:DB8::42`) are placeholders, they are IP addresses reserved for documentation by IANA (_[RFC-5737 (IPv4)][rfc-5737] and [RFC-3849 (IPv6)][rfc-3849]_). Replace them with the IP addresses you want DMS to send mail through.
+[docker-docs::network-driver::host]: https://docs.docker.com/engine/network/drivers/host
+[docker-docs::network-driver::macvlan]: https://docs.docker.com/engine/network/drivers/macvlan
+[docker-docs::compose-config::network-mode]: https://docs.docker.com/reference/compose-file/services/#network_mode
+[docker-docs::compose-config::networks]: https://docs.docker.com/reference/compose-file/services/#networks
 
 [docs::overrides-postfix]: ../../config/advanced/override-defaults/postfix.md
 [postfix-docs::smtp-bind-address-ipv4]: https://www.postfix.org/postconf.5.html#smtp_bind_address

mailserver.env

@@ -89,10 +89,10 @@ TLS_LEVEL=
 # Configures the handling of creating mails with forged sender addresses.
 #
 # **0** => (not recommended) Mail address spoofing allowed. Any logged in user may create email messages with a forged sender address (see also https://en.wikipedia.org/wiki/Email_spoofing).
-# 1 => Mail spoofing denied. Each user may only send with their own or their alias addresses. Addresses with extension delimiters(http://www.postfix.org/postconf.5.html#recipient_delimiter) are not able to send messages.
+# 1 => Mail spoofing denied. Each user may only send with his own or his alias addresses. Addresses with extension delimiters(http://www.postfix.org/postconf.5.html#recipient_delimiter) are not able to send messages.
 SPOOF_PROTECTION=
 
-# Enables the Sender Rewriting Scheme. SRS is needed if your mail server acts as forwarder. See [postsrsd](https://github.com/roehling/postsrsd/blob/main/README.rst) for further explanation.
+# Enables the Sender Rewriting Scheme. SRS is needed if your mail server acts as forwarder. See [postsrsd](https://github.com/roehling/postsrsd/blob/master/README.md#sender-rewriting-scheme-crash-course) for further explanation.
 #  - **0** => Disabled
 #  - 1 => Enabled
 ENABLE_SRS=0
@@ -508,7 +508,7 @@ DOVECOT_MAILBOX_FORMAT=maildir
 
 # empty => no
 # yes => Allow bind authentication for LDAP
-# https://doc.dovecot.org/2.4.0/core/config/auth/databases/ldap.html#authentication-bind
+# https://wiki.dovecot.org/AuthDatabase/LDAP/AuthBinds
 DOVECOT_AUTH_BIND=
 
 # -----------------------------------------------

target/bin/delmailuser

@@ -19,11 +19,7 @@ function _main() {
   for MAIL_ACCOUNT in "${@}"; do
     _account_should_already_exist
 
-    if [[ ${MAILDEL} -eq 1 ]]; then
+    [[ ${MAILDEL} -eq 1 ]] && _remove_maildir "${MAIL_ACCOUNT}"
-      _remove_maildir "${MAIL_ACCOUNT}"
-    else
-      _log 'info' "The mailbox data will not be deleted."
-    fi
 
     _manage_virtual_aliases_delete '_' "${MAIL_ACCOUNT}" \
       || _exit_with_error "Aliases for '${MAIL_ACCOUNT}' could not be deleted"
@@ -35,7 +31,7 @@ function _main() {
     _manage_accounts_delete "${MAIL_ACCOUNT}" \
       || _exit_with_error "'${MAIL_ACCOUNT}' could not be deleted"
 
-    _log 'info' "'${MAIL_ACCOUNT}' and associated data (aliases, quotas) deleted"
+    _log 'info' "'${MAIL_ACCOUNT}' and associated data deleted"
   done
 }
 
@@ -47,14 +43,14 @@ ${ORANGE}USAGE${RESET}
 
 ${ORANGE}OPTIONS${RESET}
     -y
-        Skip prompt by approving to ${LWHITE}delete all mail data${RESET} for the account(s).
+        Skip prompt by approving to ${LWHITE}delete all mail storage${RESET} for the account(s).
 
     ${BLUE}Generic Program Information${RESET}
         help       Print the usage information.
 
 ${ORANGE}DESCRIPTION${RESET}
     Delete a mail account, including associated data (aliases, quotas) and
-    optionally the mailbox data for that account.
+    optionally the mailbox storage for that account.
 
 ${ORANGE}EXAMPLES${RESET}
     ${LWHITE}./setup.sh email del user@example.com${RESET}
@@ -91,10 +87,12 @@ function _parse_options() {
 function _maildel_request_if_missing() {
   if [[ ${MAILDEL} -eq 0 ]]; then
     local MAILDEL_CHOSEN
-    read -r -p "Do you want to delete the mailbox data as well (removing all mails)? [y/N] " MAILDEL_CHOSEN
+    read -r -p "Do you want to delete the mailbox as well (removing all mails)? [Y/n] " MAILDEL_CHOSEN
 
-    # Delete mailbox data only if the user provides explicit confirmation.
+    # TODO: Why would MAILDEL be set to true if MAILDEL_CHOSEN is empty?
-    [[ ${MAILDEL_CHOSEN,,} == "y" ]] && MAILDEL=1
+    if [[ ${MAILDEL_CHOSEN} =~ (y|Y|yes|Yes) ]] || [[ -z ${MAILDEL_CHOSEN} ]]; then
+      MAILDEL=1
+    fi
   fi
 }
 
@@ -105,10 +103,10 @@ function _remove_maildir() {
   local DOMAIN_PART="${MAIL_ACCOUNT#*@}"
   local MAIL_ACCOUNT_STORAGE_DIR="/var/mail/${DOMAIN_PART}/${LOCAL_PART}"
 
-  [[ ! -d ${MAIL_ACCOUNT_STORAGE_DIR} ]] && _exit_with_error "Mailbox data directory '${MAIL_ACCOUNT_STORAGE_DIR}' does not exist"
+  [[ ! -d ${MAIL_ACCOUNT_STORAGE_DIR} ]] && _exit_with_error "Mailbox directory '${MAIL_ACCOUNT_STORAGE_DIR}' does not exist"
 
-  _log 'info' "Deleting mailbox data: '${MAIL_ACCOUNT_STORAGE_DIR}'"
+  _log 'info' "Deleting Mailbox: '${MAIL_ACCOUNT_STORAGE_DIR}'"
-  rm -R "${MAIL_ACCOUNT_STORAGE_DIR}" || _exit_with_error 'Mailbox data could not be deleted'
+  rm -R "${MAIL_ACCOUNT_STORAGE_DIR}" || _exit_with_error 'Mailbox could not be deleted'
   # Remove parent directory too if it's empty:
   rmdir "/var/mail/${DOMAIN_PART}" &>/dev/null
 }

target/bin/open-dkim

@@ -12,17 +12,9 @@ if [[ -f /etc/dms-settings ]] && [[ $(_get_dms_env_value 'ENABLE_RSPAMD') -eq 1
   fi
 fi
 
-function _main() {
+KEYSIZE=2048
-  # Default parameters (updated by `_parse_arguments()`):
+SELECTOR=mail
-  local KEYSIZE=2048
+DOMAINS=
-  local SELECTOR=mail
-  local DMS_DOMAINS=
-
-  _require_n_parameters_or_print_usage 0 "${@}"
-  _parse_arguments "${@}"
-
-  _generate_dkim_keys
-}
 
 function __usage() {
   printf '%s' "${PURPLE}OPEN-DKIM${RED}(${YELLOW}8${RED})
@@ -65,171 +57,122 @@ ${ORANGE}EXAMPLES${RESET}
 
 ${ORANGE}EXIT STATUS${RESET}
     Exit status is 0 if command was successful. If wrong arguments are provided or arguments contain
-    errors, the script will exit early with a non-zero exit status.
+    errors, the script will exit early with exit status 2.
 
 "
 }
 
-function _parse_arguments() {
+_require_n_parameters_or_print_usage 0 "${@}"
-  # Parse the command args through iteration:
-  while [[ ${#} -gt 0 ]]; do
-    case "${1}" in
 
-      ( 'keysize' )
+while [[ ${#} -gt 0 ]]; do
-        if [[ -n ${2:-} ]]; then
+  case "${1}" in
-          KEYSIZE="${2}"
+    ( 'keysize' )
-          _log 'trace' "Keysize set to '${KEYSIZE}'"
+      if [[ -n ${2+set} ]]; then
-        else
+        KEYSIZE="${2}"
-          _exit_with_error "No keysize provided after 'keysize' argument"
+        shift
-        fi
+        shift
-        ;;
+      else
+        _exit_with_error "No keysize provided after 'keysize' argument"
+      fi
+      ;;
 
-      ( 'selector' )
+    ( 'selector' )
-        if [[ -n ${2:-} ]]; then
+      if [[ -n ${2+set} ]]; then
-          SELECTOR="${2}"
+        # shellcheck disable=SC2034
-          _log 'trace' "Selector set to '${SELECTOR}'"
+        SELECTOR="${2}"
-        else
+        shift
-          _exit_with_error "No selector provided after 'selector' argument"
+        shift
-        fi
+      else
-        ;;
+        _exit_with_error "No selector provided after 'selector' argument"
+      fi
+      ;;
 
-      ( 'domain' )
+    ( 'domain' )
-        if [[ -n ${2:-} ]]; then
+      if [[ -n ${2+set} ]]; then
-          DMS_DOMAINS="${2}"
+        DOMAINS="${2}"
-          _log 'trace' "Domain(s) set to '${DMS_DOMAINS}'"
+        shift
-        else
+        shift
-          _exit_with_error "No domain(s) provided after 'domain' argument"
+      else
-        fi
+        _exit_with_error "No domain(s) provided after 'domain' argument"
-        ;;
+      fi
+      ;;
 
-      ( 'help' )
+    ( * )
-          __usage
+      __usage
-          exit 0
+      _exit_with_error "Unknown options '${1}' ${2:+and \'${2}\'}"
-          ;;
+      ;;
 
-      ( * )
+  esac
-        __usage
+done
-        _exit_with_error "Unknown option(s) ${*}"
-        ;;
-    esac
 
-    # Discard these two args (option + value) now that they've been processed:
-    shift 2
-  done
-}
-
-function _generate_dkim_keys() {
-  _generate_domains_config
-  if [[ ! -s ${DATABASE_VHOST} ]]; then
-    _log 'warn' 'No entries found, no keys to make'
-    exit 0
-  fi
-
-  # Initialize OpenDKIM configs if necessary:
-  _create_opendkim_configs
-
-  # Generate a keypair per domain and add reference to OpenDKIM configs:
-  local ENTRY_KEY KEY_TABLE_ENTRY SIGNING_TABLE_ENTRY
-  while read -r DKIM_DOMAIN; do
-    _create_dkim_key "${DKIM_DOMAIN}"
-
-    # Create / Update OpenDKIM configs with new DKIM key:
-    ENTRY_KEY="${SELECTOR}._domainkey.${DKIM_DOMAIN}"
-    KEY_TABLE_ENTRY="${ENTRY_KEY} ${DKIM_DOMAIN}:${SELECTOR}:/etc/opendkim/keys/${DKIM_DOMAIN}/${SELECTOR}.private"
-    SIGNING_TABLE_ENTRY="*@${DKIM_DOMAIN} ${ENTRY_KEY}"
-
-    # If no existing entry, add one:
-    if ! grep -q "${KEY_TABLE_ENTRY}" "${KEY_TABLE_FILE}"; then
-      echo "${KEY_TABLE_ENTRY}" >> "${KEY_TABLE_FILE}"
-    fi
-    if ! grep -q "${SIGNING_TABLE_ENTRY}" "${SIGNING_TABLE_FILE}"; then
-      echo "${SIGNING_TABLE_ENTRY}" >> "${SIGNING_TABLE_FILE}"
-    fi
-  done < <(_get_valid_lines_from_file "${DATABASE_VHOST}")
-
-  # No longer needed, remove:
-  rm "${DATABASE_VHOST}"
-
-  # Ensure ownership is consistent for all content belonging to the base directory,
-  # During container startup, an internal copy will be made via `_setup_opendkim()`
-  # with ownership we expect, while this chown is for the benefit of the users ownership.
-  chown -R "$(stat -c '%U:%G' "${OPENDKIM_BASE_DIR}")" "${OPENDKIM_BASE_DIR}"
-}
-
-# Prepare a file with one domain per line (iterated via while loop as DKIM_DOMAIN):
-# Depends on methods from `scripts/helpers/postfix.sh`:
 DATABASE_VHOST='/tmp/vhost.dkim'
+# Prepare a file with one domain per line:
 function _generate_domains_config() {
   local TMP_VHOST='/tmp/vhost.dkim.tmp'
 
   # Generate the default vhost (equivalent to /etc/postfix/vhost),
-  # unless CLI arg DMS_DOMAINS provided an alternative list to use instead:
+  # unless CLI arg DOMAINS provided an alternative list to use instead:
-  if [[ -z ${DMS_DOMAINS:-} ]]; then
+  if [[ -z ${DOMAINS} ]]; then
     _obtain_hostname_and_domainname
     # uses TMP_VHOST:
     _vhost_collect_postfix_domains
   else
-    tr ',' '\n' <<< "${DMS_DOMAINS}" >"${TMP_VHOST}"
+    tr ',' '\n' <<< "${DOMAINS}" >"${TMP_VHOST}"
   fi
 
-  # Uses DATABASE_VHOST + TMP_VHOST:
+  # uses DATABASE_VHOST + TMP_VHOST:
   _create_vhost
 }
 
-# `opendkim-genkey` generates two files at the configured `--directory`:
+_generate_domains_config
-# - <selector>.private (Private key, PEM encoded)
+if [[ ! -s ${DATABASE_VHOST} ]]; then
-# - <selector>.txt (Public key, formatted as a TXT record for a RFC 1035 DNS Zone file)
+  _log 'warn' 'No entries found, no keys to make'
-function _create_dkim_key() {
+  exit 0
-  DKIM_DOMAIN=${1?Expected to be provided a domain}
+fi
 
-  OPENDKIM_DOMAINKEY_DIR="${OPENDKIM_BASE_DIR}/keys/${DKIM_DOMAIN}"
+while read -r DKIM_DOMAIN; do
-  mkdir -p "${OPENDKIM_DOMAINKEY_DIR}"
+  mkdir -p "/tmp/docker-mailserver/opendkim/keys/${DKIM_DOMAIN}"
 
-  DKIM_KEY_FILE="${OPENDKIM_DOMAINKEY_DIR}/${SELECTOR}.private"
+  if [[ ! -f "/tmp/docker-mailserver/opendkim/keys/${DKIM_DOMAIN}/${SELECTOR}.private" ]]; then
-  if [[ ! -f "${DKIM_KEY_FILE}" ]]; then
+    _log 'info' "Creating DKIM private key '/tmp/docker-mailserver/opendkim/keys/${DKIM_DOMAIN}/${SELECTOR}.private'"
-    _log 'info' "Creating DKIM private key '${DKIM_KEY_FILE}'"
 
-    # NOTE:
-    # --domain only affects a comment in the generated DNS Zone file
-    # --subdomains is the default,
-    # --nosubdomains would add `t=s` to the DNS TXT record generated
-    # http://www.opendkim.org/opendkim-genkey.8.html
     opendkim-genkey \
       --bits="${KEYSIZE}" \
       --subdomains \
       --domain="${DKIM_DOMAIN}" \
       --selector="${SELECTOR}" \
-      --directory="${OPENDKIM_DOMAINKEY_DIR}"
+      --directory="/tmp/docker-mailserver/opendkim/keys/${DKIM_DOMAIN}"
   fi
-}
 
-OPENDKIM_BASE_DIR='/tmp/docker-mailserver/opendkim'
+  # fix permissions to use the same user:group as /tmp/docker-mailserver/opendkim/keys
-KEY_TABLE_FILE="${OPENDKIM_BASE_DIR}/KeyTable"
+  chown -R "$(stat -c '%U:%G' /tmp/docker-mailserver/opendkim/keys)" "/tmp/docker-mailserver/opendkim/keys/${DKIM_DOMAIN}"
-SIGNING_TABLE_FILE="${OPENDKIM_BASE_DIR}/SigningTable"
-TRUSTED_HOSTS_FILE="${OPENDKIM_BASE_DIR}/TrustedHosts"
-# Create configs if missing:
-function _create_opendkim_configs() {
-  mkdir -p "${OPENDKIM_BASE_DIR}"
-  local OPENDKIM_CONFIGS=(
-    "${KEY_TABLE_FILE}"
-    "${SIGNING_TABLE_FILE}"
-    "${TRUSTED_HOSTS_FILE}"
-  )
 
-  # Only create if the file doesn't exist (avoids modifying mtime):
+  # write to KeyTable if necessary
-  for FILE in "${OPENDKIM_CONFIGS[@]}"; do
+  KEYTABLEENTRY="${SELECTOR}._domainkey.${DKIM_DOMAIN} ${DKIM_DOMAIN}:${SELECTOR}:/etc/opendkim/keys/${DKIM_DOMAIN}/${SELECTOR}.private"
-    if [[ ! -f "${FILE}" ]]; then
+  if [[ ! -f "/tmp/docker-mailserver/opendkim/KeyTable" ]]; then
-      _log 'debug' "Creating OpenDKIM config '${FILE}'"
+    _log 'debug' 'Creating DKIM KeyTable'
-      touch "${FILE}"
+    echo "${KEYTABLEENTRY}" >/tmp/docker-mailserver/opendkim/KeyTable
+  else
+    if ! grep -q "${KEYTABLEENTRY}" "/tmp/docker-mailserver/opendkim/KeyTable"; then
+      echo "${KEYTABLEENTRY}" >>/tmp/docker-mailserver/opendkim/KeyTable
     fi
-  done
-
-  # If file exists but is empty, add default hosts to trust:
-  if [[ ! -s "${TRUSTED_HOSTS_FILE}" ]]; then
-    _log 'debug' 'Adding default trust to OpenDKIM TrustedHosts config'
-    echo "127.0.0.1" >  "${TRUSTED_HOSTS_FILE}"
-    echo "localhost" >> "${TRUSTED_HOSTS_FILE}"
   fi
-}
 
-_main "${@}"
+  # write to SigningTable if necessary
+  SIGNINGTABLEENTRY="*@${DKIM_DOMAIN} ${SELECTOR}._domainkey.${DKIM_DOMAIN}"
+  if [[ ! -f /tmp/docker-mailserver/opendkim/SigningTable ]]; then
+    _log 'debug' 'Creating DKIM SigningTable'
+    echo "*@${DKIM_DOMAIN} ${SELECTOR}._domainkey.${DKIM_DOMAIN}" >/tmp/docker-mailserver/opendkim/SigningTable
+  else
+    if ! grep -q "${SIGNINGTABLEENTRY}" /tmp/docker-mailserver/opendkim/SigningTable; then
+      echo "${SIGNINGTABLEENTRY}" >>/tmp/docker-mailserver/opendkim/SigningTable
+    fi
+  fi
+done < <(_get_valid_lines_from_file "${DATABASE_VHOST}")
+
+# create TrustedHosts if missing
+if [[ -d /tmp/docker-mailserver/opendkim ]] && [[ ! -f /tmp/docker-mailserver/opendkim/TrustedHosts ]]; then
+  _log 'debug' 'Creating DKIM TrustedHosts'
+  echo "127.0.0.1" >/tmp/docker-mailserver/opendkim/TrustedHosts
+  echo "localhost" >>/tmp/docker-mailserver/opendkim/TrustedHosts
+fi

target/bin/setup

@@ -64,7 +64,6 @@ ${RED}[${ORANGE}SUB${RED}]${ORANGE}COMMANDS${RESET}
 
     ${LBLUE}COMMAND${RESET} debug ${RED}:=${RESET}
         setup debug ${CYAN}fetchmail${RESET}
-        setup debug ${CYAN}getmail${RESET}
         setup debug ${CYAN}login${RESET} <COMMANDS>
         setup debug ${CYAN}show-mail-logs${RESET}
 
@@ -151,7 +150,6 @@ function _main() {
     ( debug )
       case ${2:-} in
         ( fetchmail      ) debug-fetchmail ;;
-        ( getmail        ) debug-getmail ;;
         ( show-mail-logs ) cat /var/log/mail/mail.log ;;
         ( login          )
           shift 2

target/postfix/main.cf

@@ -68,10 +68,9 @@ smtpd_forbid_bare_newline = yes
 # smtpd_forbid_bare_newline_exclusions = $mynetworks
 
 # Custom defined parameters for DMS:
-# Custom sender restrictions overview: https://github.com/docker-mailserver/docker-mailserver/pull/4379#issuecomment-2670365917
+# reject_unknown_sender_domain: https://github.com/docker-mailserver/docker-mailserver/issues/3716#issuecomment-1868033234
-# `reject_unknown_sender_domain`: https://github.com/docker-mailserver/docker-mailserver/issues/3716#issuecomment-1868033234
 dms_smtpd_sender_restrictions = permit_sasl_authenticated, permit_mynetworks, reject_unknown_sender_domain
-# `SPOOF_PROTECTION=1` support requires prepending `reject_authenticated_sender_login_mismatch`
+# Submission ports 587 and 465 support for SPOOF_PROTECTION=1
 mua_sender_restrictions = reject_authenticated_sender_login_mismatch, $dms_smtpd_sender_restrictions
 
 # Postscreen settings to drop zombies/open relays/spam early

target/postfix/sender_header_filter.pcre

@@ -8,5 +8,4 @@
 /^\s*X-Mailer/                      IGNORE
 /^\s*X-Originating-IP/              IGNORE
 /^\s*Received: from.*127.0.0.1/     IGNORE
-/^\s*X-MS-Reactions:/               IGNORE
+/^Content-Type:/i                   PREPEND X-MS-Reactions: disallow
-/^\s*Message-Id:/i                  PREPEND X-MS-Reactions: disallow

target/scripts/build/compile.sh

@@ -11,39 +11,28 @@ source /usr/local/bin/helpers/log.sh
 # shellcheck disable=SC2310
 _log_level_is 'trace' && QUIET='-y' || QUIET='-qq'
 
-function _install_build_deps() {
+function _compile_dovecot_fts_xapian() {
   apt-get "${QUIET}" update
   apt-get "${QUIET}" install --no-install-recommends \
     automake libtool pkg-config libicu-dev libsqlite3-dev libxapian-dev make build-essential dh-make devscripts dovecot-dev
-}
 
-function _build_package() {
+  local XAPIAN_VERSION='1.7.13'
-  local XAPIAN_VERSION='1.9'
+  curl -sSfL -o dovecot-fts-xapian.tar.gz \
-  curl -fsSL "https://github.com/grosjo/fts-xapian/releases/download/${XAPIAN_VERSION}/dovecot-fts-xapian-${XAPIAN_VERSION}.tar.gz" \
+    "https://github.com/grosjo/fts-xapian/releases/download/${XAPIAN_VERSION}/dovecot-fts-xapian-${XAPIAN_VERSION}.tar.gz"
-    | tar -xz
+  tar xf dovecot-fts-xapian.tar.gz
+
   cd "fts-xapian-${XAPIAN_VERSION}"
+  USER=root dh_make -p "dovecot-fts-xapian-${XAPIAN_VERSION}" --single --native --copyright gpl2 -y
 
-  # Prepare for building DEB source package:
-  # https://manpages.debian.org/bookworm/dh-make/dh_make.1.en.html
-  # License LGPL 2.1: https://github.com/grosjo/fts-xapian/issues/174#issuecomment-2422404568
-  USER=root dh_make --packagename "dovecot-fts-xapian-${XAPIAN_VERSION}" --single --native --copyright lgpl2 -y
-  # Remove generated example files:
   rm debian/*.ex
-  # Add required package metadata:
+  cp PACKAGES/DEB/control debian/
-  # https://www.debian.org/doc/manuals/maint-guide/dreq.en.html#control
+  cp PACKAGES/DEB/changelog debian/
-  curl -fsSL https://raw.githubusercontent.com/grosjo/fts-xapian/refs/tags/1.7.16/PACKAGES/DEB/control > debian/control
+  cp PACKAGES/DEB/compat debian/
-  # Replace version number:
-  sed -i -E "s|(dovecot-fts-xapian)-[1-9\.-]+|\1-${XAPIAN_VERSION}|g" debian/control
-  # Required to proceed with debuild:
-  # https://www.debian.org/doc/manuals/maint-guide/dother.en.html#compat
-  # (13 is the default debhelper version from the original `dh_make` generated `debian/control`):
-  echo '13' > debian/compat
 
-  # Build arch specific binary package via debuild:
+  sed -i -E "s|(dovecot-fts-xapian)-[1-9\.-]+|\1-${XAPIAN_VERSION}|g" debian/control
-  # https://manpages.debian.org/bookworm/devscripts/debuild.1.en.html
+  sed -i -E "s|(dovecot-fts-xapian)-[1-9\.-]+ \(.*\)(.*)|\1-${XAPIAN_VERSION} (${XAPIAN_VERSION})\2|g" debian/changelog
-  # https://manpages.debian.org/bookworm/dpkg-dev/dpkg-buildpackage.1.en.html
+
-  debuild --no-sign --build=any | tee /tmp/debuild.log 2>&1
+  debuild -us -uc -B | tee /tmp/debuild.log 2>&1
 }
 
-_install_build_deps
+_compile_dovecot_fts_xapian
-_build_package

target/scripts/build/packages.sh

@@ -24,58 +24,31 @@ function _pre_installation_steps() {
   apt-get "${QUIET}" upgrade
 
   _log 'trace' 'Installing packages that are needed early'
-  # Add packages usually required by apt to:
+  # add packages usually required by apt to
+  # - not log unnecessary warnings
+  # - be able to add PPAs early (e.g., Rspamd)
   local EARLY_PACKAGES=(
-    # Avoid logging unnecessary warnings:
+    apt-utils # avoid useless warnings
-    apt-utils
+    apt-transport-https ca-certificates curl gnupg # required for adding PPAs
-    # Required for adding third-party repos (/etc/apt/sources.list.d) as alternative package sources (eg: Dovecot CE and Rspamd):
+    systemd-standalone-sysusers # avoid problems with SA / Amavis (https://github.com/docker-mailserver/docker-mailserver/pull/3403#pullrequestreview-1596689953)
-    apt-transport-https ca-certificates curl gnupg
-    # Avoid problems with SA / Amavis (https://github.com/docker-mailserver/docker-mailserver/pull/3403#pullrequestreview-1596689953):
-    systemd-standalone-sysusers
   )
   apt-get "${QUIET}" install --no-install-recommends "${EARLY_PACKAGES[@]}" 2>/dev/null
 }
 
-# Install third-party commands to /usr/local/bin
 function _install_utils() {
-  local ARCH_A
-  ARCH_A=$(uname --machine)
-  # Alternate naming convention support: x86_64 (amd64) / aarch64 (arm64)
-  # https://en.wikipedia.org/wiki/X86-64#Industry_naming_conventions
-  local ARCH_B
-  case "${ARCH_A}" in
-    ( 'x86_64'  ) ARCH_B='amd64' ;;
-    ( 'aarch64' ) ARCH_B='arm64' ;;
-    ( * )
-      _log 'error' "Unsupported arch: '${ARCH_A}'"
-      return 1
-      ;;
-  esac
-
-  # TIP: `*.tar.gz` releases tend to forget to reset UID/GID ownership when archiving.
-  # When extracting with `tar` as `root` the archived UID/GID is kept, unless using `--no-same-owner`.
-  # Likewise when the binary is in a nested location the full archived path
-  # must be provided + `--strip-components` to extract the file to the target directory.
-  # Doing this avoids the need for (`mv` + `rm`) or (`--to-stdout` + `chmod +x`)
   _log 'debug' 'Installing utils sourced from Github'
-
   _log 'trace' 'Installing jaq'
-  local JAQ_TAG='v2.1.0'
+  local JAQ_TAG='v2.0.0'
-  curl -sSfL "https://github.com/01mf02/jaq/releases/download/${JAQ_TAG}/jaq-$(uname -m)-unknown-linux-gnu" -o /usr/local/bin/jaq
+  curl -sSfL "https://github.com/01mf02/jaq/releases/download/${JAQ_TAG}/jaq-$(uname -m)-unknown-linux-gnu" -o /usr/bin/jaq
-  chmod +x /usr/local/bin/jaq
+  chmod +x /usr/bin/jaq
-
-  _log 'trace' 'Installing step'
-  local STEP_RELEASE='0.28.2'
-  curl -sSfL "https://github.com/smallstep/cli/releases/download/v${STEP_RELEASE}/step_linux_${STEP_RELEASE}_${ARCH_B}.tar.gz" \
-    | tar -xz --directory /usr/local/bin --no-same-owner --strip-components=2 "step_${STEP_RELEASE}/bin/step"
 
   _log 'trace' 'Installing swaks'
-  # `perl-doc` is required for `swaks --help` to work:
   apt-get "${QUIET}" install --no-install-recommends perl-doc
   local SWAKS_VERSION='20240103.0'
   local SWAKS_RELEASE="swaks-${SWAKS_VERSION}"
-  curl -sSfL "https://github.com/jetmore/swaks/releases/download/v${SWAKS_VERSION}/${SWAKS_RELEASE}.tar.gz" \
+  curl -sSfL "https://github.com/jetmore/swaks/releases/download/v${SWAKS_VERSION}/${SWAKS_RELEASE}.tar.gz" | tar -xz
-    | tar -xz --directory /usr/local/bin --no-same-owner --strip-components=1 "${SWAKS_RELEASE}/swaks"
+  mv "${SWAKS_RELEASE}/swaks" /usr/local/bin
+  rm -r "${SWAKS_RELEASE}"
 }
 
 function _install_postfix() {
@@ -163,56 +136,43 @@ function _install_dovecot() {
     dovecot-pop3d dovecot-sieve
   )
 
-  # Additional Dovecot packages for supporting the DMS community (docs-only guide contributions).
+  # Dovecot packages for community supported features.
   DOVECOT_PACKAGES+=(dovecot-auth-lua)
 
-  # (Opt-in via ENV) Change repo source for dovecot packages to a third-party repo maintained by Dovecot.
+  # Dovecot's deb community repository only provides x86_64 packages, so do not include it
-  # NOTE: AMD64 / x86_64 is the only supported arch from the Dovecot CE repo (thus noDMS built for ARM64 / aarch64)
+  # when building for another architecture.
-  # Repo: https://repo.dovecot.org/ce-2.4-latest/debian/bookworm/dists/bookworm/main/
-  # Docs: https://repo.dovecot.org/#debian
   if [[ ${DOVECOT_COMMUNITY_REPO} -eq 1 ]] && [[ "$(uname --machine)" == "x86_64" ]]; then
-    # WARNING: Repo only provides Debian Bookworm package support for Dovecot CE 2.4+.
+    _log 'trace' 'Using Dovecot community repository'
-    # As Debian Bookworm only packages Dovecot 2.3.x, building DMS with this alternative package repo may not yet be compatible with DMS:
+    curl -sSfL https://repo.dovecot.org/DOVECOT-REPO-GPG | gpg --import
-    # - 2.3.19: https://salsa.debian.org/debian/dovecot/-/tree/stable/bookworm
+    gpg --export ED409DA1 > /etc/apt/trusted.gpg.d/dovecot.gpg
-    # - 2.3.21: https://salsa.debian.org/debian/dovecot/-/tree/stable/bookworm-backports
+    echo "deb https://repo.dovecot.org/ce-2.3-latest/debian/${VERSION_CODENAME} ${VERSION_CODENAME} main" > /etc/apt/sources.list.d/dovecot.list
 
-    _log 'trace' 'Adding third-party package repository (Dovecot)'
+    _log 'trace' 'Updating Dovecot package signatures'
-    curl -fsSL https://repo.dovecot.org/DOVECOT-REPO-GPG-2.4 | gpg --dearmor > /usr/share/keyrings/upstream-dovecot.gpg
-    echo \
-      "deb [signed-by=/usr/share/keyrings/upstream-dovecot.gpg] https://repo.dovecot.org/ce-2.4-latest/debian/${VERSION_CODENAME} ${VERSION_CODENAME} main" \
-      > /etc/apt/sources.list.d/upstream-dovecot.list
-
-    # Refresh package index:
     apt-get "${QUIET}" update
 
-    # This repo instead provides `dovecot-auth-lua` as a transitional package to `dovecot-lua`,
+    # Additional community package needed for Lua support if the Dovecot community repository is used.
-    # thus this extra package is required to retain lua support:
     DOVECOT_PACKAGES+=(dovecot-lua)
   fi
 
   _log 'debug' 'Installing Dovecot'
   apt-get "${QUIET}" install --no-install-recommends "${DOVECOT_PACKAGES[@]}"
 
-  # Runtime dependency for fts_xapian (built via `compile.sh`):
+  # dependency for fts_xapian
   apt-get "${QUIET}" install --no-install-recommends libxapian30
 }
 
 function _install_rspamd() {
-  # NOTE: DMS only supports the rspamd package via using the third-party repo maintained by Rspamd (AMD64 + ARM64):
+  _log 'debug' 'Installing Rspamd'
-  # Repo: https://rspamd.com/apt-stable/dists/bookworm/main/
+  _log 'trace' 'Adding Rspamd PPA'
-  # Docs: https://rspamd.com/downloads.html#debian-and-ubuntu-linux
+  curl -sSfL https://rspamd.com/apt-stable/gpg.key | gpg --dearmor >/etc/apt/trusted.gpg.d/rspamd.gpg
-  # NOTE: Debian 12 provides Rspamd 3.4 (too old) and Rspamd discourages it's use
-
-  _log 'trace' 'Adding third-party package repository (Rspamd)'
-  curl -fsSL https://rspamd.com/apt-stable/gpg.key | gpg --dearmor > /usr/share/keyrings/upstream-rspamd.gpg
   echo \
-    "deb [signed-by=/usr/share/keyrings/upstream-rspamd.gpg] https://rspamd.com/apt-stable/ ${VERSION_CODENAME} main" \
+    "deb [signed-by=/etc/apt/trusted.gpg.d/rspamd.gpg] http://rspamd.com/apt-stable/ ${VERSION_CODENAME} main" \
-    > /etc/apt/sources.list.d/upstream-rspamd.list
+    >/etc/apt/sources.list.d/rspamd.list
 
-  # Refresh package index:
+  _log 'trace' 'Updating package index after adding PPAs'
   apt-get "${QUIET}" update
 
-  _log 'debug' 'Installing Rspamd'
+  _log 'trace' 'Installing actual package'
   apt-get "${QUIET}" install rspamd redis-server
 }
 

target/scripts/helpers/change-detection.sh

@@ -43,7 +43,7 @@ function _monitored_files_checksums() {
 
     # Check whether Rspamd is used and if so, monitor it's changes as well
     if [[ ${ENABLE_RSPAMD} -eq 1 ]] && [[ -d ${RSPAMD_DMS_D} ]]; then
-      readarray -d '' STAGING_FILES_RSPAMD < <(find "${RSPAMD_DMS_D}" -type f -print0)
+      readarray -d '' STAGING_FILES_RSPAMD < <(find "${RSPAMD_DMS_D}" -type f -name "*.sh" -print0)
       STAGING_FILES+=("${STAGING_FILES_RSPAMD[@]}")
     fi
   fi

target/scripts/helpers/error.sh

@@ -1,7 +1,7 @@
 #!/bin/bash
 
 function _exit_with_error() {
-  if [[ -n ${1:-} ]]; then
+  if [[ -n ${1+set} ]]; then
     _log 'error' "${1}"
   else
     _log 'error' "Call to '_exit_with_error' is missing a message to log"

target/scripts/helpers/log.sh

@@ -43,12 +43,12 @@ RESET=$(echo -ne   '\e[0m')
 # message is logged. Likewise when the second argument
 # is missing. Both failures will return with exit code '1'.
 function _log() {
-  if [[ -z ${1:-} ]]; then
+  if [[ -z ${1+set} ]]; then
     _log 'error' "Call to '_log' is missing a valid log level"
     return 1
   fi
 
-  if [[ -z ${2:-} ]]; then
+  if [[ -z ${2+set} ]]; then
     _log 'error' "Call to '_log' is missing a message to log"
     return 1
   fi
@@ -116,7 +116,7 @@ function _log() {
 # variables file. If this does not yield a value either,
 # use the default log level.
 function _get_log_level_or_default() {
-  if [[ -n ${LOG_LEVEL:-} ]]; then
+  if [[ -n ${LOG_LEVEL+set} ]]; then
     echo "${LOG_LEVEL}"
   elif [[ -e /etc/dms-settings ]] && grep -q -E "^LOG_LEVEL='[a-z]+'" /etc/dms-settings; then
     grep '^LOG_LEVEL=' /etc/dms-settings | cut -d "'" -f 2

target/scripts/helpers/rspamd.sh

@@ -111,6 +111,14 @@ function _rspamd_handle_user_modules_adjustments() {
     fi
   }
 
+  # We check for usage of the previous location of the commands file.
+  # TODO This can be removed after the release of v14.0.0.
+  local RSPAMD_DMS_CUSTOM_COMMANDS_F_OLD="${RSPAMD_DMS_D}-modules.conf"
+  readonly RSPAMD_DMS_CUSTOM_COMMANDS_F_OLD
+  if [[ -f ${RSPAMD_DMS_CUSTOM_COMMANDS_F_OLD} ]]; then
+    _dms_panic__general "Old custom command file location '${RSPAMD_DMS_CUSTOM_COMMANDS_F_OLD}' is deprecated (use '${RSPAMD_DMS_CUSTOM_COMMANDS_F}' now)" 'Rspamd setup'
+  fi
+
   if [[ -f "${RSPAMD_DMS_CUSTOM_COMMANDS_F}" ]]; then
     __rspamd__log 'debug' "Found file '${RSPAMD_DMS_CUSTOM_COMMANDS_F}' - parsing and applying it"
 

target/scripts/helpers/utils.sh

@@ -122,7 +122,7 @@ function _reload_postfix() {
 # you can set the environment variable `POSTFIX_README_DIRECTORY='/new/dir/'`
 # (`POSTFIX_` is an arbitrary prefix, you can choose the one you like),
 # and then call this function:
-# `_replace_by_env_in_file 'POSTFIX_' '<PATH TO POSTFIX's main.cf>`
+# `_replace_by_env_in_file 'POSTFIX_' 'PATH TO POSTFIX's main.cf>`
 #
 # ## Panics
 #
@@ -131,9 +131,9 @@ function _reload_postfix() {
 # 1. No first and second argument is supplied
 # 2. The second argument is a path to a file that does not exist
 function _replace_by_env_in_file() {
-  if [[ -z ${1:-} ]]; then
+  if [[ -z ${1+set} ]]; then
     _dms_panic__invalid_value 'first argument unset' 'utils.sh:_replace_by_env_in_file'
-  elif [[ -z ${2:-} ]]; then
+  elif [[ -z ${2+set} ]]; then
     _dms_panic__invalid_value 'second argument unset' 'utils.sh:_replace_by_env_in_file'
   elif [[ ! -f ${2} ]]; then
     _dms_panic__invalid_value "file '${2}' does not exist" 'utils.sh:_replace_by_env_in_file'

target/scripts/start-mailserver.sh

@@ -38,11 +38,13 @@ function _register_functions() {
   # ? >> Checks
 
   _register_check_function '_check_hostname'
+  _register_check_function '_check_log_level'
   _register_check_function '_check_spam_prefix'
 
   # ? >> Setup
 
   _register_setup_function '_setup_vmail_id'
+  _register_setup_function '_setup_logs_general'
   _register_setup_function '_setup_timezone'
 
   if [[ ${SMTP_ONLY} -ne 1 ]]; then
@@ -61,6 +63,7 @@ function _register_functions() {
       ;;
 
     ( 'LDAP' )
+      _environment_variables_ldap
       _register_setup_function '_setup_ldap'
       ;;
 
@@ -73,8 +76,15 @@ function _register_functions() {
       ;;
   esac
 
-  [[ ${ENABLE_OAUTH2} -eq 1 ]] && _register_setup_function '_setup_oauth2'
+  if [[ ${ENABLE_OAUTH2} -eq 1 ]]; then
-  [[ ${ENABLE_SASLAUTHD} -eq 1 ]] && _register_setup_function '_setup_saslauthd'
+      _environment_variables_oauth2
+      _register_setup_function '_setup_oauth2'
+  fi
+
+  if [[ ${ENABLE_SASLAUTHD} -eq 1 ]]; then
+    _environment_variables_saslauthd
+    _register_setup_function '_setup_saslauthd'
+  fi
 
   _register_setup_function '_setup_dovecot_inet_protocols'
 
@@ -88,6 +98,7 @@ function _register_functions() {
   _register_setup_function '_setup_ssl'
   _register_setup_function '_setup_docker_permit'
   _register_setup_function '_setup_mailname'
+  _register_setup_function '_setup_dovecot_hostname'
 
   _register_setup_function '_setup_postfix_early'
 
@@ -111,23 +122,20 @@ function _register_functions() {
   _register_setup_function '_setup_logwatch'
 
   _register_setup_function '_setup_save_states'
-  _register_setup_function '_setup_adjust_state_permissions'
+  _register_setup_function '_setup_apply_fixes_after_configuration'
+  _register_setup_function '_environment_variables_export'
 
   if [[ ${ENABLE_MTA_STS} -eq 1 ]]; then
     _register_setup_function '_setup_mta_sts'
     _register_start_daemon '_start_daemon_mta_sts_daemon'
   fi
 
-  # ! The following functions must be executed after all other setup functions
-  _register_setup_function '_setup_directory_and_file_permissions'
-  _register_setup_function '_setup_run_user_patches'
-
   # ? >> Daemons
 
   _register_start_daemon '_start_daemon_cron'
   _register_start_daemon '_start_daemon_rsyslog'
 
-  [[ ${SMTP_ONLY} -ne 1 ]] && _register_start_daemon '_start_daemon_dovecot'
+  [[ ${SMTP_ONLY}               -ne 1 ]] && _register_start_daemon '_start_daemon_dovecot'
 
   if [[ ${ENABLE_UPDATE_CHECK} -eq 1 ]]; then
     if [[ ${DMS_RELEASE} != 'edge' ]]; then
@@ -166,27 +174,26 @@ function _register_functions() {
 # ? >> Executing all stacks / actual start of DMS
 # ------------------------------------------------------------
 
-_early_supervisor_setup
-_early_variables_setup
-
-_log 'info' "Welcome to docker-mailserver ${DMS_RELEASE}"
-
-_register_functions
-_check
-
 # Ensure DMS only adjusts config files for a new container.
 # Container restarts should skip as they retain the modified config.
-if [[ -f /CONTAINER_START ]]; then
+if [[ ! -f /CONTAINER_START ]]; then
-  _log 'info' 'Container was restarted. Skipping most setup routines.'
+  _early_supervisor_setup
-  # We cannot skip all setup routines because some need to run _after_
+  _early_variables_setup
-  # the initial setup (and hence, they cannot be moved to the check stack).
-  _setup_directory_and_file_permissions
 
-  # shellcheck source=./startup/setup.d/mail_state.sh
+  _log 'info' "Welcome to docker-mailserver ${DMS_RELEASE}"
-  source /usr/local/bin/setup.d/mail_state.sh
+
-  _setup_adjust_state_permissions
+  _register_functions
-else
+  _check
   _setup
+  _run_user_patches
+else
+  # container was restarted
+  _early_variables_setup
+
+  _log 'info' 'Container was restarted. Skipping setup routines.'
+  _log 'info' "Welcome to docker-mailserver ${DMS_RELEASE}"
+
+  _register_functions
 fi
 
 # marker to check if container was restarted

target/scripts/startup/check-stack.sh

@@ -26,6 +26,24 @@ function _check_hostname() {
   fi
 }
 
+function _check_log_level() {
+  if [[ ${LOG_LEVEL} == 'trace' ]] \
+  || [[ ${LOG_LEVEL} == 'debug' ]] \
+  || [[ ${LOG_LEVEL} == 'info' ]]  \
+  || [[ ${LOG_LEVEL} == 'warn' ]]  \
+  || [[ ${LOG_LEVEL} == 'error' ]]
+  then
+    return 0
+  else
+    local DEFAULT_LOG_LEVEL='info'
+    _log 'warn' "Log level '${LOG_LEVEL}' is invalid (falling back to default '${DEFAULT_LOG_LEVEL}')"
+
+    # shellcheck disable=SC2034
+    VARS[LOG_LEVEL]="${DEFAULT_LOG_LEVEL}"
+    LOG_LEVEL="${DEFAULT_LOG_LEVEL}"
+  fi
+}
+
 function _check_spam_prefix() {
   # This check should be independent of ENABLE_POP3 and ENABLE_IMAP
   if [[ ${MOVE_SPAM_TO_JUNK} -eq 0 ]] \

target/scripts/startup/setup-stack.sh

@@ -82,9 +82,7 @@ function _setup_timezone() {
   fi
 }
 
-# Misc checks and fixes migrated here until next refactor:
+function _setup_apply_fixes_after_configuration() {
-# NOTE: `start-mailserver.sh` runs this along with `mail-state.sh` during container restarts
-function _setup_directory_and_file_permissions() {
   _log 'trace' 'Removing leftover PID files from a stop/start'
   find /var/run/ -not -name 'supervisord.pid' -name '*.pid' -delete
   touch /dev/shm/supervisor.sock
@@ -103,11 +101,9 @@ function _setup_directory_and_file_permissions() {
     _log 'debug' "Ensuring '${RSPAMD_DMS_DKIM_D}' is owned by '_rspamd:_rspamd'"
     chown -R _rspamd:_rspamd "${RSPAMD_DMS_DKIM_D}"
   fi
-
-  __log_fixes
 }
 
-function _setup_run_user_patches() {
+function _run_user_patches() {
   local USER_PATCHES='/tmp/docker-mailserver/user-patches.sh'
 
   if [[ -f ${USER_PATCHES} ]]; then
@@ -117,32 +113,3 @@ function _setup_run_user_patches() {
     _log 'trace' "No optional '${USER_PATCHES}' provided"
   fi
 }
-
-function __log_fixes() {
-  _log 'debug' 'Ensuring /var/log/mail owneership + permissions are correct'
-
-  # File/folder permissions are fine when using docker volumes, but may be wrong
-  # when file system folders are mounted into the container.
-  # Set the expected values and create missing folders/files just in case.
-  mkdir -p /var/log/{mail,supervisor}
-
-  # TODO: Remove these lines in a future release once concerns are resolved:
-  # https://github.com/docker-mailserver/docker-mailserver/pull/4370#issuecomment-2661762043
-  chown syslog:root /var/log/mail
-
-  if [[ ${ENABLE_CLAMAV} -eq 1 ]]; then
-    # TODO: Consider assigning /var/log/mail a writable non-root group for other processes like ClamAV?
-    # - Check if ClamAV is capable of creating files itself when they're missing?
-    # - Alternatively a symlink to /var/log/mail from the original intended location would allow write access
-    #   as a user to the symlink location, while keeping ownership as root at /var/log/mail
-    # - `LogSyslog false` for clamd.conf + freshclam.conf could possibly be enabled instead of log files?
-    #   However without better filtering in place (once Vector is adopted), this should be avoided.
-    touch /var/log/mail/{clamav,freshclam}.log
-    chown clamav:adm /var/log/mail/{clamav,freshclam}.log
-  fi
-
-  # Volume permissions should be corrected:
-  # https://github.com/docker-mailserver/docker-mailserver-helm/issues/137
-  chmod 755 /var/log/mail/
-  find /var/log/mail/ -type f -exec chmod 640 {} +
-}

target/scripts/startup/setup.d/dmarc_dkim_spf.sh

@@ -23,11 +23,7 @@ function _setup_opendkim() {
     # check if any keys are available
     if [[ -e /tmp/docker-mailserver/opendkim/KeyTable ]]; then
       cp -a /tmp/docker-mailserver/opendkim/* /etc/opendkim/
-
+      _log 'trace' "DKIM keys added for: $(find /etc/opendkim/keys/ -maxdepth 1 -type f -printf '%f ')"
-      local DKIM_DOMAINS
-      DKIM_DOMAINS=$(find /etc/opendkim/keys/ -maxdepth 1 -type f -printf '%f ')
-      _log 'trace' "DKIM keys added for: ${DKIM_DOMAINS}"
-
       chown -R opendkim:opendkim /etc/opendkim/
       chmod -R 0700 /etc/opendkim/keys/
     else

target/scripts/startup/setup.d/dovecot.sh

@@ -3,54 +3,13 @@
 function _setup_dovecot() {
   _log 'debug' 'Setting up Dovecot'
 
-  # Protocol support
-  sedfile -i -e 's|include_try /usr/share/dovecot/protocols.d|include_try /etc/dovecot/protocols.d|g' /etc/dovecot/dovecot.conf
   cp -a /usr/share/dovecot/protocols.d /etc/dovecot/
-  # Disable these protocols by default, they can be enabled later via ENV (ENABLE_POP3, ENABLE_IMAP, ENABLE_MANAGESIEVE)
+  # disable pop3 (it will be eventually enabled later in the script, if requested)
   mv /etc/dovecot/protocols.d/pop3d.protocol /etc/dovecot/protocols.d/pop3d.protocol.disab
+  # disable imap (it will be eventually enabled later in the script, if requested)
   mv /etc/dovecot/protocols.d/imapd.protocol /etc/dovecot/protocols.d/imapd.protocol.disab
   mv /etc/dovecot/protocols.d/managesieved.protocol /etc/dovecot/protocols.d/managesieved.protocol.disab
-
+    sedfile -i 's|^postmaster_address = .*$|postmaster_address = '"${POSTMASTER_ADDRESS}"'|g' /etc/dovecot/conf.d/15-lda.conf
-  # NOTE: While Postfix will deliver to Dovecot via LMTP (Previously LDA until DMS v2),
-  # LDA may be used via other services like Getmail being configured to use /usr/lib/dovecot/deliver
-  # when mail does not need to go through Postfix.
-  # `mail_plugins` is scoped to the `protocol lda` config block of this file.
-  #
-  # TODO: `postmaster_address` + `hostname` appear to be for the general Dovecot config rather than LDA specific?
-  # https://doc.dovecot.org/2.3/settings/core/#core_setting-postmaster_address
-  # https://doc.dovecot.org/2.3/settings/core/#core_setting-hostname
-  # Dovecot 3.0 docs:
-  # https://doc.dovecot.org/main/core/summaries/settings.html#postmaster_address
-  # https://doc.dovecot.org/main/core/summaries/settings.html#postmaster_address
-  # https://doc.dovecot.org/main/core/config/delivery/lmtp.html#common-delivery-settings
-  # https://doc.dovecot.org/main/core/config/delivery/lda.html#common-delivery-settings
-  # https://doc.dovecot.org/main/core/config/sieve/submission.html#postmaster-address
-  # Shows config example with postmaster_address scoped in a `protocol lda { }` block:
-  # https://doc.dovecot.org/main/howto/virtual/simple_install.html#delivering-mails
-  #
-  # DMS initially copied Dovecot example configs, these were removed from Dovecot 2.4 onwards:
-  # https://github.com/dovecot/core/commit/5941699b277d762d98c202928cf5b5c8c70bc359
-  # In favor of a minimal config example:
-  # https://github.com/dovecot/core/commit/9a6a6aef35bb403fa96f0b5efdb0faff85b1471d
-  # 2.3 series example config:
-  # https://github.com/dovecot/core/blob/2.3.21.1/doc/example-config/conf.d/15-lda.conf
-  # Initial config files committed to DMS in April 2016:
-  # TODO: Consider housekeeping on config to only represent relevant changes/support by scripts
-  # https://github.com/docker-mailserver/docker-mailserver/commit/ee0d0853dd672488238eecb0ec2d26719ff45d7d
-  #
-  # TODO: `mail_plugins` appending `sieve` should probably be done for both `15-lda.conf` and `20-lmtp.conf`
-  # Presently DMS replaces the `20-lmtp.conf` from `dovecot-lmtpd` package with our own modified copy from 2016.
-  # The DMS variant only makes this one change to that file, thus we could adjust it as we do below for `15-lda.conf`
-  # Reference: https://github.com/docker-mailserver/docker-mailserver/pull/4350#issuecomment-2646736328
-
-  # shellcheck disable=SC2016
-  sedfile -i -r \
-    -e 's|^(\s*)#?(mail_plugins =).*|\1\2 $mail_plugins sieve|' \
-    -e 's|^#?(lda_mailbox_autocreate =).*|\1 yes|' \
-    -e 's|^#?(lda_mailbox_autosubscribe =).*|\1 yes|' \
-    -e "s|^#?(postmaster_address =).*|\1 ${POSTMASTER_ADDRESS}|" \
-    -e "s|^#?(hostname =).*|\1 ${HOSTNAME}|" \
-    /etc/dovecot/conf.d/15-lda.conf
 
   if ! grep -q -E '^stats_writer_socket_path=' /etc/dovecot/dovecot.conf; then
     printf '\n%s\n' 'stats_writer_socket_path=' >>/etc/dovecot/dovecot.conf
@@ -65,7 +24,6 @@ function _setup_dovecot() {
       sedfile -i -E \
         "s|^(mail_location =).*|\1 ${DOVECOT_MAILBOX_FORMAT}:/var/mail/%d/%n|" \
         /etc/dovecot/conf.d/10-mail.conf
-
       _log 'trace' 'Enabling cron job for dbox purge'
       mv /etc/cron.d/dovecot-purge.disabled /etc/cron.d/dovecot-purge
       chmod 644 /etc/cron.d/dovecot-purge
@@ -97,12 +55,6 @@ function _setup_dovecot() {
   [[ -f /tmp/docker-mailserver/dovecot.cf ]] && cp /tmp/docker-mailserver/dovecot.cf /etc/dovecot/local.conf
 }
 
-# The `sieve` plugin is always enabled in DMS, this method handles user supplied sieve scripts + ManageSieve protocol
-# NOTE: There is a related post-setup step for this sieve support handled at `_setup_post()` (setup-stack.sh)
-# TODO: Improved sieve support may be needed in DMS to support this use-case:
-# https://github.com/docker-mailserver/docker-mailserver/issues/3904
-# TODO: Change detection support + refactor/DRY this sieve logic:
-# https://github.com/orgs/docker-mailserver/discussions/2633#discussioncomment-11622955
 function _setup_dovecot_sieve() {
   mkdir -p /usr/lib/dovecot/sieve-{filter,global,pipe}
   mkdir -p /usr/lib/dovecot/sieve-global/{before,after}
@@ -240,3 +192,8 @@ function _setup_dovecot_inet_protocols() {
 function _setup_dovecot_dhparam() {
   _setup_dhparam 'Dovecot' '/etc/dovecot/dh.pem'
 }
+
+function _setup_dovecot_hostname() {
+  _log 'debug' 'Applying hostname to Dovecot'
+  sedfile -i "s|^#hostname =.*$|hostname = '${HOSTNAME}'|g" /etc/dovecot/conf.d/15-lda.conf
+}

target/scripts/startup/setup.d/log.sh

@@ -1,5 +1,15 @@
 #!/bin/bash
 
+function _setup_logs_general() {
+  _log 'debug' 'Setting up general log files'
+
+  # File/folder permissions are fine when using docker volumes, but may be wrong
+  # when file system folders are mounted into the container.
+  # Set the expected values and create missing folders/files just in case.
+  mkdir -p /var/log/{mail,supervisor}
+  chown syslog:root /var/log/mail
+}
+
 function _setup_logrotate() {
   _log 'debug' 'Setting up logrotate'
 

target/scripts/startup/setup.d/mail_state.sh

@@ -1,138 +1,125 @@
 #!/bin/bash
 
-DMS_STATE_DIR='/var/mail-state'
-
 # Consolidate all states into a single directory
 # (/var/mail-state) to allow persistence using docker volumes
 function _setup_save_states() {
-  if [[ ! -d ${DMS_STATE_DIR} ]]; then
+  local DEST DESTDIR STATEDIR SERVICEDIR SERVICEDIRS SERVICEFILE SERVICEFILES
-    _log 'debug' "'${DMS_STATE_DIR}' is not present - not consolidating state"
+
-    return 0
+  STATEDIR='/var/mail-state'
+
+  if [[ -d ${STATEDIR} ]]; then
+    _log 'debug' "Consolidating all state onto ${STATEDIR}"
+
+    # Always enabled features:
+    SERVICEDIRS=(
+      lib/logrotate
+      lib/postfix
+      spool/postfix
+    )
+
+    # Only consolidate state for services that are enabled
+    # Notably avoids copying over 200MB for the ClamAV database
+    [[ ${ENABLE_AMAVIS}       -eq 1 ]] && SERVICEDIRS+=('lib/amavis')
+    [[ ${ENABLE_CLAMAV}       -eq 1 ]] && SERVICEDIRS+=('lib/clamav')
+    [[ ${ENABLE_FAIL2BAN}     -eq 1 ]] && SERVICEDIRS+=('lib/fail2ban')
+    [[ ${ENABLE_FETCHMAIL}    -eq 1 ]] && SERVICEDIRS+=('lib/fetchmail')
+    [[ ${ENABLE_GETMAIL}      -eq 1 ]] && SERVICEDIRS+=('lib/getmail')
+    [[ ${ENABLE_MTA_STS}      -eq 1 ]] && SERVICEDIRS+=('lib/mta-sts')
+    [[ ${ENABLE_POSTGREY}     -eq 1 ]] && SERVICEDIRS+=('lib/postgrey')
+    [[ ${ENABLE_RSPAMD}       -eq 1 ]] && SERVICEDIRS+=('lib/rspamd')
+    [[ ${ENABLE_RSPAMD_REDIS} -eq 1 ]] && SERVICEDIRS+=('lib/redis')
+    [[ ${ENABLE_SPAMASSASSIN} -eq 1 ]] && SERVICEDIRS+=('lib/spamassassin')
+    [[ ${ENABLE_SRS}          -eq 1 ]] && SERVICEDIRS+=('lib/postsrsd')
+    [[ ${SMTP_ONLY}           -ne 1 ]] && SERVICEDIRS+=('lib/dovecot')
+
+    # Single service files
+    [[ ${ENABLE_SRS}          -eq 1 ]] && SERVICEFILES+=('/etc/postsrsd.secret')
+
+    for SERVICEFILE in "${SERVICEFILES[@]}"; do
+      DEST="${STATEDIR}/${SERVICEFILE}"
+      DESTDIR="${DEST%/*}"
+
+      mkdir -p "${DESTDIR}"
+      if [[ -f ${DEST} ]]; then
+        _log 'trace' "Destination ${DEST} exists, linking ${SERVICEFILE} to it"
+        # Original content from image no longer relevant, remove it:
+        rm -f "${SERVICEFILE}"
+      elif [[ -f "${SERVICEFILE}" ]]; then
+        _log 'trace' "Moving ${SERVICEFILE} to ${DEST}"
+        # Empty volume was mounted, or new content from enabling a feature ENV:
+        mv "${SERVICEFILE}" "${DEST}"
+        # Apply SELinux security context to match the state directory, so access
+        # is not restricted to the current running container:
+        chcon -R --reference="${STATEDIR}" "${DEST}" 2>/dev/null || true
+      fi
+
+      # Symlink the original file in the container ($SERVICEFILE) to be
+      # sourced from assocaiated path in /var/mail-state/ ($DEST):
+      ln -s "${DEST}" "${SERVICEFILE}"
+    done
+
+    for SERVICEDIR in "${SERVICEDIRS[@]}"; do
+      DEST="${STATEDIR}/${SERVICEDIR//\//-}"
+      SERVICEDIR="/var/${SERVICEDIR}"
+
+      # If relevant content is found in /var/mail-state (presumably a volume mount),
+      # use it instead. Otherwise copy over any missing directories checked.
+      if [[ -d ${DEST} ]]; then
+        _log 'trace' "Destination ${DEST} exists, linking ${SERVICEDIR} to it"
+        # Original content from image no longer relevant, remove it:
+        rm -rf "${SERVICEDIR}"
+      elif [[ -d ${SERVICEDIR} ]]; then
+        _log 'trace' "Moving contents of ${SERVICEDIR} to ${DEST}"
+        # An empty volume was mounted, or new content dir now exists from enabling a feature ENV:
+        mv "${SERVICEDIR}" "${DEST}"
+        # Apply SELinux security context to match the state directory, so access
+        # is not restricted to the current running container:
+        chcon -R --reference="${STATEDIR}" "${DEST}" 2>/dev/null || true
+      else
+        _log 'error' "${SERVICEDIR} should exist but is missing"
+      fi
+
+      # Symlink the original path in the container ($SERVICEDIR) to be
+      # sourced from assocaiated path in /var/mail-state/ ($DEST):
+      ln -s "${DEST}" "${SERVICEDIR}"
+    done
+
+    # This ensures the user and group of the files from the external mount have their
+    # numeric ID values in sync. New releases where the installed packages order changes
+    # can change the values in the Docker image, causing an ownership mismatch.
+    # NOTE: More details about users and groups added during image builds are documented here:
+    # https://github.com/docker-mailserver/docker-mailserver/pull/3011#issuecomment-1399120252
+    _log 'trace' "Fixing ${STATEDIR}/* permissions"
+    [[ ${ENABLE_AMAVIS}       -eq 1 ]] && chown -R amavis:amavis             "${STATEDIR}/lib-amavis"
+    [[ ${ENABLE_CLAMAV}       -eq 1 ]] && chown -R clamav:clamav             "${STATEDIR}/lib-clamav"
+    [[ ${ENABLE_FETCHMAIL}    -eq 1 ]] && chown -R fetchmail:nogroup         "${STATEDIR}/lib-fetchmail"
+    [[ ${ENABLE_MTA_STS}      -eq 1 ]] && chown -R _mta-sts:_mta-sts         "${STATEDIR}/lib-mta-sts"
+    [[ ${ENABLE_POSTGREY}     -eq 1 ]] && chown -R postgrey:postgrey         "${STATEDIR}/lib-postgrey"
+    [[ ${ENABLE_RSPAMD}       -eq 1 ]] && chown -R _rspamd:_rspamd           "${STATEDIR}/lib-rspamd"
+    [[ ${ENABLE_RSPAMD_REDIS} -eq 1 ]] && chown -R redis:redis               "${STATEDIR}/lib-redis"
+    [[ ${ENABLE_SPAMASSASSIN} -eq 1 ]] && chown -R debian-spamd:debian-spamd "${STATEDIR}/lib-spamassassin"
+
+    chown -R root:root "${STATEDIR}/lib-logrotate"
+    chown -R postfix:postfix "${STATEDIR}/lib-postfix"
+
+    # NOTE: The Postfix spool location has mixed owner/groups to take into account:
+    # UID = postfix(101): active, bounce, corrupt, defer, deferred, flush, hold, incoming, maildrop, private, public, saved, trace
+    # UID = root(0): dev, etc, lib, pid, usr
+    # GID = postdrop(103): maildrop, public
+    # GID for all other directories is root(0)
+    # NOTE: `spool-postfix/private/` will be set to `postfix:postfix` when Postfix starts / restarts
+    # Set most common ownership:
+    chown -R postfix:root "${STATEDIR}/spool-postfix"
+    chown root:root "${STATEDIR}/spool-postfix"
+
+    # These two require the postdrop(103) group:
+    chgrp -R postdrop "${STATEDIR}"/spool-postfix/{maildrop,public}
+
+    # These permissions rely on the `postdrop` binary having the SGID bit set.
+    # Ref: https://github.com/docker-mailserver/docker-mailserver/pull/3625
+    chmod 730 "${STATEDIR}/spool-postfix/maildrop"
+    chmod 710 "${STATEDIR}/spool-postfix/public"
+  else
+    _log 'debug' "'${STATEDIR}' is not present; Not consolidating state"
   fi
-
-  _log 'debug' "Consolidating all state onto ${DMS_STATE_DIR}"
-
-  local DEST SERVICEDIR SERVICEDIRS SERVICEFILE SERVICEFILES
-
-  # Always enabled features:
-  SERVICEDIRS=(
-    'lib/logrotate'
-    'lib/postfix'
-    'spool/postfix'
-  )
-
-  # Only consolidate state for services that are enabled
-  # Notably avoids copying over 200MB for the ClamAV database
-  [[ ${ENABLE_AMAVIS}       -eq 1 ]] && SERVICEDIRS+=('lib/amavis')
-  [[ ${ENABLE_CLAMAV}       -eq 1 ]] && SERVICEDIRS+=('lib/clamav')
-  [[ ${ENABLE_FAIL2BAN}     -eq 1 ]] && SERVICEDIRS+=('lib/fail2ban')
-  [[ ${ENABLE_FETCHMAIL}    -eq 1 ]] && SERVICEDIRS+=('lib/fetchmail')
-  [[ ${ENABLE_GETMAIL}      -eq 1 ]] && SERVICEDIRS+=('lib/getmail')
-  [[ ${ENABLE_MTA_STS}      -eq 1 ]] && SERVICEDIRS+=('lib/mta-sts')
-  [[ ${ENABLE_POSTGREY}     -eq 1 ]] && SERVICEDIRS+=('lib/postgrey')
-  [[ ${ENABLE_RSPAMD}       -eq 1 ]] && SERVICEDIRS+=('lib/rspamd')
-  [[ ${ENABLE_RSPAMD_REDIS} -eq 1 ]] && SERVICEDIRS+=('lib/redis')
-  [[ ${ENABLE_SPAMASSASSIN} -eq 1 ]] && SERVICEDIRS+=('lib/spamassassin')
-  [[ ${ENABLE_SRS}          -eq 1 ]] && SERVICEDIRS+=('lib/postsrsd')
-  [[ ${SMTP_ONLY}           -ne 1 ]] && SERVICEDIRS+=('lib/dovecot')
-
-  # Single service files
-  [[ ${ENABLE_SRS}          -eq 1 ]] && SERVICEFILES+=('/etc/postsrsd.secret')
-
-  for SERVICEFILE in "${SERVICEFILES[@]}"; do
-    DEST="${DMS_STATE_DIR}/${SERVICEFILE}"
-
-    # Append service parent dir(s) path to the state dir and ensure it exists:
-    mkdir -p "${DEST%/*}"
-    if [[ -f ${DEST} ]]; then
-      _log 'trace' "Destination ${DEST} exists, linking ${SERVICEFILE} to it"
-      # Original content from image no longer relevant, remove it:
-      rm -f "${SERVICEFILE}"
-    elif [[ -f "${SERVICEFILE}" ]]; then
-      _log 'trace' "Moving ${SERVICEFILE} to ${DEST}"
-      # Empty volume was mounted, or new content from enabling a feature ENV:
-      mv "${SERVICEFILE}" "${DEST}"
-      # Apply SELinux security context to match the state directory, so access
-      # is not restricted to the current running container:
-      chcon -R --reference="${DMS_STATE_DIR}" "${DEST}" 2>/dev/null || true
-    fi
-
-    # Symlink the original file in the container ($SERVICEFILE) to be
-    # sourced from assocaiated path in /var/mail-state/ ($DEST):
-    ln -s "${DEST}" "${SERVICEFILE}"
-  done
-
-  for SERVICEDIR in "${SERVICEDIRS[@]}"; do
-    DEST="${DMS_STATE_DIR}/${SERVICEDIR//\//-}"
-    SERVICEDIR="/var/${SERVICEDIR}"
-
-    # If relevant content is found in /var/mail-state (presumably a volume mount),
-    # use it instead. Otherwise copy over any missing directories checked.
-    if [[ -d ${DEST} ]]; then
-      _log 'trace' "Destination ${DEST} exists, linking ${SERVICEDIR} to it"
-      # Original content from image no longer relevant, remove it:
-      rm -rf "${SERVICEDIR}"
-    elif [[ -d ${SERVICEDIR} ]]; then
-      _log 'trace' "Moving contents of ${SERVICEDIR} to ${DEST}"
-      # An empty volume was mounted, or new content dir now exists from enabling a feature ENV:
-      mv "${SERVICEDIR}" "${DEST}"
-      # Apply SELinux security context to match the state directory, so access
-      # is not restricted to the current running container:
-      # https://github.com/docker-mailserver/docker-mailserver/pull/3890
-      chcon -R --reference="${DMS_STATE_DIR}" "${DEST}" 2>/dev/null || true
-    else
-      _log 'error' "${SERVICEDIR} should exist but is missing"
-    fi
-
-    # Symlink the original path in the container ($SERVICEDIR) to be
-    # sourced from associated path in /var/mail-state/ ($DEST):
-    ln -s "${DEST}" "${SERVICEDIR}"
-  done
-}
-
-# These corrections are to fix changes to UID/GID values between upgrades,
-# or when ownership/permissions were altered externally on the host (eg: migration or system scripts)
-function _setup_adjust_state_permissions() {
-  [[ ! -d ${DMS_STATE_DIR} ]] && return 0
-
-  # Parent directories must have executable bit set to descend the file tree for access,
-  # as each service running as a non-root user requires this to access their state directory,
-  # `/var/mail-state` must allow all users `+x`:
-  chmod +x "${DMS_STATE_DIR}"
-
-  # This ensures the user and group of the files from the external mount have their
-  # numeric ID values in sync. New releases where the installed packages order changes
-  # can change the values in the Docker image, causing an ownership mismatch.
-  # NOTE: More details about users and groups added during image builds are documented here:
-  # https://github.com/docker-mailserver/docker-mailserver/pull/3011#issuecomment-1399120252
-  _log 'trace' "Ensuring correct ownership + permissions for DMS state dir: '${DMS_STATE_DIR}'"
-  [[ ${ENABLE_AMAVIS}       -eq 1 ]] && chown -R amavis:amavis             "${DMS_STATE_DIR}/lib-amavis"
-  [[ ${ENABLE_CLAMAV}       -eq 1 ]] && chown -R clamav:clamav             "${DMS_STATE_DIR}/lib-clamav"
-  [[ ${ENABLE_FETCHMAIL}    -eq 1 ]] && chown -R fetchmail:nogroup         "${DMS_STATE_DIR}/lib-fetchmail"
-  [[ ${ENABLE_MTA_STS}      -eq 1 ]] && chown -R _mta-sts:_mta-sts         "${DMS_STATE_DIR}/lib-mta-sts"
-  [[ ${ENABLE_POSTGREY}     -eq 1 ]] && chown -R postgrey:postgrey         "${DMS_STATE_DIR}/lib-postgrey"
-  [[ ${ENABLE_RSPAMD}       -eq 1 ]] && chown -R _rspamd:_rspamd           "${DMS_STATE_DIR}/lib-rspamd"
-  [[ ${ENABLE_RSPAMD_REDIS} -eq 1 ]] && chown -R redis:redis               "${DMS_STATE_DIR}/lib-redis"
-  [[ ${ENABLE_SPAMASSASSIN} -eq 1 ]] && chown -R debian-spamd:debian-spamd "${DMS_STATE_DIR}/lib-spamassassin"
-
-  chown -R root:root "${DMS_STATE_DIR}/lib-logrotate"
-  chown -R postfix:postfix "${DMS_STATE_DIR}/lib-postfix"
-
-  # NOTE: The Postfix spool location has mixed owner/groups to take into account:
-  # UID = postfix(101): active, bounce, corrupt, defer, deferred, flush, hold, incoming, maildrop, private, public, saved, trace
-  # UID = root(0): dev, etc, lib, pid, usr
-  # GID = postdrop(103): maildrop, public
-  # GID for all other directories is root(0)
-  # NOTE: `spool-postfix/private/` will be set to `postfix:postfix` when Postfix starts / restarts
-  # Set most common ownership:
-  chown -R postfix:root "${DMS_STATE_DIR}/spool-postfix"
-  chown root:root "${DMS_STATE_DIR}/spool-postfix"
-
-  # These two require the postdrop(103) group:
-  chgrp -R postdrop "${DMS_STATE_DIR}"/spool-postfix/{maildrop,public}
-
-  # These permissions rely on the `postdrop` binary having the SGID bit set.
-  # Ref: https://github.com/docker-mailserver/docker-mailserver/pull/3625
-  chmod 730 "${DMS_STATE_DIR}/spool-postfix/maildrop"
-  chmod 710 "${DMS_STATE_DIR}/spool-postfix/public"
 }

target/scripts/startup/setup.d/postfix.sh

@@ -93,19 +93,13 @@ EOF
 function _setup_postfix_late() {
   _log 'debug' 'Configuring Postfix (late setup)'
 
-  # These two config files are `access` database tables managed via `setup email restrict`:
-  # NOTE: Prepends to existing restrictions, thus has priority over other permit/reject policies that follow.
-  # https://www.postfix.org/postconf.5.html#smtpd_sender_restrictions
-  # https://www.postfix.org/access.5.html
   __postfix__log 'trace' 'Configuring user access'
   if [[ -f /tmp/docker-mailserver/postfix-send-access.cf ]]; then
-    # Prefer to prepend to our specialized variant instead:
+    sed -i -E 's|(smtpd_sender_restrictions =)|\1 check_sender_access texthash:/tmp/docker-mailserver/postfix-send-access.cf,|' /etc/postfix/main.cf
-    # https://github.com/docker-mailserver/docker-mailserver/pull/4379
-    sed -i -E 's|^(dms_smtpd_sender_restrictions =)|\1 check_sender_access texthash:/tmp/docker-mailserver/postfix-send-access.cf,|' /etc/postfix/main.cf
   fi
 
   if [[ -f /tmp/docker-mailserver/postfix-receive-access.cf ]]; then
-    sed -i -E 's|^(smtpd_recipient_restrictions =)|\1 check_recipient_access texthash:/tmp/docker-mailserver/postfix-receive-access.cf,|' /etc/postfix/main.cf
+    sed -i -E 's|(smtpd_recipient_restrictions =)|\1 check_recipient_access texthash:/tmp/docker-mailserver/postfix-receive-access.cf,|' /etc/postfix/main.cf
   fi
 
   __postfix__log 'trace' 'Configuring relay host'

target/scripts/startup/setup.d/security/misc.sh

@@ -155,6 +155,13 @@ function __setup__security__clamav() {
   if [[ ${ENABLE_CLAMAV} -eq 1 ]]; then
     _log 'debug' 'Enabling and configuring ClamAV'
 
+    local FILE
+    for FILE in /var/log/mail/{clamav,freshclam}.log; do
+      touch "${FILE}"
+      chown clamav:adm "${FILE}"
+      chmod 640 "${FILE}"
+    done
+
     if [[ ${CLAMAV_MESSAGE_SIZE_LIMIT} != '25M' ]]; then
       _log 'trace' "Setting ClamAV message scan size limit to '${CLAMAV_MESSAGE_SIZE_LIMIT}'"
 

target/scripts/startup/variables-stack.sh

@@ -4,16 +4,9 @@
 declare -A VARS
 
 function _early_variables_setup() {
-  __environment_variables_log_level
   _obtain_hostname_and_domainname
   __environment_variables_backwards_compatibility
   __environment_variables_general_setup
-
-  [[ ${ACCOUNT_PROVISIONER} == 'LDAP' ]] && __environment_variables_ldap
-  [[ ${ENABLE_OAUTH2} -eq 1 ]]           && __environment_variables_oauth2
-  [[ ${ENABLE_SASLAUTHD} -eq 1 ]]        && __environment_variables_saslauthd
-
-  __environment_variables_export
 }
 
 # This function handles variables that are deprecated. This allows a
@@ -62,8 +55,6 @@ function __environment_variables_general_setup() {
   VARS[DMS_VMAIL_UID]="${DMS_VMAIL_UID:=5000}"
   VARS[DMS_VMAIL_GID]="${DMS_VMAIL_GID:=5000}"
 
-  # user-customizable are last
-
   _log 'trace' 'Setting anti-spam & anti-virus environment variables'
 
   VARS[AMAVIS_LOGLEVEL]="${AMAVIS_LOGLEVEL:=0}"
@@ -168,27 +159,15 @@ function __environment_variables_general_setup() {
   VARS[UPDATE_CHECK_INTERVAL]="${UPDATE_CHECK_INTERVAL:=1d}"
 }
 
-function __environment_variables_log_level() {
+function _environment_variables_oauth2() {
-  if [[ ${LOG_LEVEL} == 'trace' ]] \
+  _log 'debug' 'Setting OAUTH2-related environment variables now'
-  || [[ ${LOG_LEVEL} == 'debug' ]] \
-  || [[ ${LOG_LEVEL} == 'info' ]]  \
-  || [[ ${LOG_LEVEL} == 'warn' ]]  \
-  || [[ ${LOG_LEVEL} == 'error' ]]
-  then
-    return 0
-  else
-    local DEFAULT_LOG_LEVEL='info'
-    _log 'warn' "Log level '${LOG_LEVEL}' is invalid (falling back to default '${DEFAULT_LOG_LEVEL}')"
 
-    # shellcheck disable=SC2034
+  VARS[OAUTH2_INTROSPECTION_URL]="${OAUTH2_INTROSPECTION_URL:=}"
-    VARS[LOG_LEVEL]="${DEFAULT_LOG_LEVEL}"
-    LOG_LEVEL="${DEFAULT_LOG_LEVEL}"
-  fi
 }
 
 # This function handles environment variables related to LDAP.
 # NOTE: SASLAuthd and Dovecot LDAP support inherit these common ENV.
-function __environment_variables_ldap() {
+function _environment_variables_ldap() {
   _log 'debug' 'Setting LDAP-related environment variables now'
 
   VARS[LDAP_BIND_DN]="${LDAP_BIND_DN:=}"
@@ -198,15 +177,9 @@ function __environment_variables_ldap() {
   VARS[LDAP_START_TLS]="${LDAP_START_TLS:=no}"
 }
 
-function __environment_variables_oauth2() {
-  _log 'debug' 'Setting OAUTH2-related environment variables now'
-
-  VARS[OAUTH2_INTROSPECTION_URL]="${OAUTH2_INTROSPECTION_URL:=}"
-}
-
 # This function handles environment variables related to SASLAUTHD
 # LDAP specific ENV handled in: `startup/setup.d/saslauthd.sh:_setup_saslauthd()`
-function __environment_variables_saslauthd() {
+function _environment_variables_saslauthd() {
   _log 'debug' 'Setting SASLAUTHD-related environment variables now'
 
   # This ENV is only used by the supervisor service config `saslauth.conf`:
@@ -217,7 +190,7 @@ function __environment_variables_saslauthd() {
 # This function Writes the contents of the `VARS` map (associative array)
 # to locations where they can be sourced from (e.g. `/etc/dms-settings`)
 # or where they can be used by Bash directly (e.g. `/root/.bashrc`).
-function __environment_variables_export() {
+function _environment_variables_export() {
   _log 'debug' "Exporting environment variables now (creating '/etc/dms-settings')"
 
   : >/root/.bashrc     # make DMS variables available in login shells and their subprocesses

test/helper/common.bash

@@ -56,7 +56,7 @@ function __handle_container_name() {
   if [[ -n ${1:-} ]] && [[ ${1:-} =~ ^dms-test_ ]]; then
     printf '%s' "${1}"
     return 0
-  elif [[ -n ${CONTAINER_NAME:-} ]]; then
+  elif [[ -n ${CONTAINER_NAME+set} ]]; then
     printf '%s' "${CONTAINER_NAME}"
     return 0
   else

test/helper/setup.bash

@@ -17,7 +17,7 @@
 # This function is internal and should not be used in tests.
 function __initialize_variables() {
   function __check_if_set() {
-    if [[ -z ${!1:-} ]]; then
+    if [[ ${!1+set} != 'set' ]]; then
       echo "ERROR: (helper/setup.sh) '${1:?No variable name given to __check_if_set}' is not set" >&2
       exit 1
     fi

test/tests/parallel/set3/container_configuration/hostname.bats

@@ -234,9 +234,8 @@ function _should_have_correct_mail_headers() {
   # but Amavis is changing that. It also changes protocol from SMTP to ESMTP.
   assert_line --index 7 --partial 'Received: from localhost (localhost [127.0.0.1])'
   assert_line --index 8 --partial "by ${EXPECTED_FQDN} (Postfix) with ESMTP id"
-  assert_line --index 14 'X-MS-Reactions: disallow'
+  assert_line --index 14 --partial 'Message-Id:'
-  assert_line --index 15 --partial 'Message-Id:'
+  assert_line --index 14 --partial "@${EXPECTED_FQDN}>"
-  assert_line --index 15 --partial "@${EXPECTED_FQDN}>"
 
   # Mail contents example:
   #

test/tests/serial/open_dkim.bats

@@ -62,7 +62,7 @@ function teardown() { _default_teardown ; }
 
   __init_container_without_waiting
 
-  __should_generate_dkim_key 7
+  __should_generate_dkim_key 6
   __assert_outputs_common_dkim_logs
 
   __should_have_tables_trustedhosts_for_domain
@@ -78,7 +78,7 @@ function teardown() { _default_teardown ; }
   # Only mount single config file (postfix-virtual.cf):
   __init_container_without_waiting "${PWD}/test/config/postfix-virtual.cf:/tmp/docker-mailserver/postfix-virtual.cf:ro"
 
-  __should_generate_dkim_key 6
+  __should_generate_dkim_key 5
   __assert_outputs_common_dkim_logs
 
   __should_have_tables_trustedhosts_for_domain
@@ -95,7 +95,7 @@ function teardown() { _default_teardown ; }
   # Only mount single config file (postfix-accounts.cf):
   __init_container_without_waiting "${PWD}/test/config/postfix-accounts.cf:/tmp/docker-mailserver/postfix-accounts.cf:ro"
 
-  __should_generate_dkim_key 6
+  __should_generate_dkim_key 5
   __assert_outputs_common_dkim_logs
 
   __should_have_tables_trustedhosts_for_domain
@@ -113,7 +113,7 @@ function teardown() { _default_teardown ; }
   __init_container_without_waiting '/tmp/docker-mailserver'
 
   # generate first key (with a custom selector)
-  __should_generate_dkim_key 5 '1024' 'domain1.tld' 'mailer'
+  __should_generate_dkim_key 4 '1024' 'domain1.tld' 'mailer'
   __assert_outputs_common_dkim_logs
   # generate two additional keys different to the previous one
   __should_generate_dkim_key 2 '1024' 'domain2.tld,domain3.tld'
@@ -183,15 +183,15 @@ function __assert_logged_dkim_creation() {
 
 function __assert_outputs_common_dkim_logs() {
   refute_output --partial 'No entries found, no keys to make'
-  assert_output --partial "Creating OpenDKIM config '/tmp/docker-mailserver/opendkim/KeyTable'"
+  assert_output --partial 'Creating DKIM KeyTable'
-  assert_output --partial "Creating OpenDKIM config '/tmp/docker-mailserver/opendkim/SigningTable'"
+  assert_output --partial 'Creating DKIM SigningTable'
-  assert_output --partial "Creating OpenDKIM config '/tmp/docker-mailserver/opendkim/TrustedHosts'"
+  assert_output --partial 'Creating DKIM TrustedHosts'
 }
 
 function __should_support_creating_key_of_size() {
   local EXPECTED_KEYSIZE=${1:-}
 
-  __should_generate_dkim_key 7 "${EXPECTED_KEYSIZE}"
+  __should_generate_dkim_key 6 "${EXPECTED_KEYSIZE}"
   __assert_outputs_common_dkim_logs
   __assert_logged_dkim_creation 'localdomain2.com'
   __assert_logged_dkim_creation 'localhost.localdomain'