base.m4, config.m4: Remove spaces from the `dkim_sign_headers' setting.
authorMark Wooding <mdw@distorted.org.uk>
Mon, 6 May 2024 03:15:06 +0000 (04:15 +0100)
committerMark Wooding <mdw@distorted.org.uk>
Mon, 6 May 2024 03:21:32 +0000 (04:21 +0100)
It turns out that Exim secretly requires there to be no spaces here.
Or, more specifically, it compares the header names in the message
against the list entries between the colons without stripping spaces, so
none of them will match.

base.m4
config.m4

diff --git a/base.m4 b/base.m4
index 7c9cdb7..799e3ee 100644 (file)
--- a/base.m4
+++ b/base.m4
@@ -451,7 +451,7 @@ m4_define(<:DKIM_SIGN:>,
                        {CONF_dkim_keys_dir/$value/active/$dkim_selector.priv}:>)
        dkim_canon = relaxed
        dkim_strict = true
-       dkim_sign_headers = CONF_dkim_headers : \
+       dkim_sign_headers = CONF_dkim_headers:\
                X-CONF_header_token-DKIM-Key-Publication
        headers_add = \
                ${if DKIM_SIGN_P \
index 5e3cc9b..7fba559 100644 (file)
--- a/config.m4
+++ b/config.m4
@@ -81,16 +81,16 @@ DEFCONF(relay_clients, <m4_dnl
 )
 
 ## DKIM headers list.
-DEFCONF(dkim_headers, m4_dnl
-References : In-Reply-To : Subject : To : Date : Message-ID : m4_dnl
-From : Sender : Reply-To : Cc : m4_dnl
-Resent-Date : Resent-From : Resent-Sender : Resent-To : Resent-Cc : m4_dnl
-Resent-Message-ID : m4_dnl
-Content-Transfer-Encoding : Content-Type : MIME-Version : m4_dnl
-Content-ID : Content-Description : m4_dnl
-List-Id : List-Help : List-Unsubscribe : List-Subscribe : m4_dnl
-List-Post : List-Owner : List-Archive m4_dnl
-)
+## Surprise!  Internal whitespace isn't allowed here.
+DEFCONF(dkim_headers,m4_dnl
+References:In-Reply-To:Subject:To:Date:Message-ID:m4_dnl
+From:Sender:Reply-To:Cc:m4_dnl
+Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:m4_dnl
+Resent-Message-ID:m4_dnl
+Content-Transfer-Encoding:Content-Type:MIME-Version:m4_dnl
+Content-ID:Content-Description:m4_dnl
+List-Id:List-Help:List-Unsubscribe:List-Subscribe:m4_dnl
+List-Post:List-Owner:List-Archive)
 
 ## TLS certificate list.
 DEFCONF(certlist,