Skip to content

Reject DKIM signatures that do not cover the whole message body#321

Merged
link2xt merged 1 commit intomainfrom
link2xt/opendkim-sig-length
Jun 18, 2024
Merged

Reject DKIM signatures that do not cover the whole message body#321
link2xt merged 1 commit intomainfrom
link2xt/opendkim-sig-length

Conversation

@link2xt
Copy link
Copy Markdown
Contributor

@link2xt link2xt commented Jun 11, 2024

Closes #297

@link2xt link2xt force-pushed the link2xt/opendkim-sig-length branch 4 times, most recently from 67b8fd9 to 567db16 Compare June 12, 2024 00:41
Copy link
Copy Markdown
Contributor

@hpk42 hpk42 left a comment

Choose a reason for hiding this comment

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

lgtm -- test would be nice but at least changelog entry needs to rebased.

@link2xt link2xt force-pushed the link2xt/opendkim-sig-length branch from 567db16 to 33059dc Compare June 14, 2024 20:01
@link2xt link2xt merged commit 57c29c1 into main Jun 18, 2024
@link2xt link2xt deleted the link2xt/opendkim-sig-length branch June 18, 2024 02:48
@link2xt
Copy link
Copy Markdown
Contributor Author

link2xt commented Jul 3, 2024

I reverted this in b1d11d7

Apparently this caused problems with outlook.com: #338

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.

Reject messages with partial DKIM signatures

2 participants