Skip to content

feat(postfix): disable processing of MIME headers#1016

Merged
link2xt merged 1 commit into
mainfrom
link2xt/disable-mime-header-processing
Jul 13, 2026
Merged

feat(postfix): disable processing of MIME headers#1016
link2xt merged 1 commit into
mainfrom
link2xt/disable-mime-header-processing

Conversation

@link2xt

@link2xt link2xt commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Default behavior is processing headers
that are actually part of the message body.
This is unlikely to be needed
as headers such as Received are not added into MIME parts, but may result in broken DKIM signatures
if the body is modified.

Default behavior is processing headers
that are actually part of the message body.
This is unlikely to be needed
as headers such as Received are not added into MIME parts,
but may result in broken DKIM signatures
if the body is modified.
@link2xt

link2xt commented Jun 30, 2026

Copy link
Copy Markdown
Contributor Author

This setting looks like what we want for simplicity in any case, as parsing nested and MIME headers is unexpected (didn't know Postfix does this).

May help with #1005 and https://support.delta.chat/t/how-can-i-access-unencrypted-emails-on-the-nine-testrun-org-server/5474 as there is one less possibility of postfix modifying the message body and breaking DKIM signatures.

@link2xt
link2xt marked this pull request as ready for review June 30, 2026 22:43

@j-g00da j-g00da left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh wow, I wonder if this causes any DKIM issues in the wild

@link2xt
link2xt merged commit deca1ef into main Jul 13, 2026
8 checks passed
@link2xt
link2xt deleted the link2xt/disable-mime-header-processing branch July 13, 2026 09:46
feld added a commit to feld/chatmail-cookbook that referenced this pull request Jul 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants