]> source.dussan.org Git - rspamd.git/commit
[Minor] Dkim_signing: correct is_skip_sign logic 3214/head
authorPeter Wu <peter@lekensteyn.nl>
Thu, 16 Jan 2020 01:11:49 +0000 (01:11 +0000)
committerPeter Wu <peter@lekensteyn.nl>
Thu, 16 Jan 2020 01:30:56 +0000 (01:30 +0000)
commit845b8c00dc03d2a98f5514eda15f0e8909cd96b7
tree0f3c350cdb3629b1cd3f7060fd4728f0bfcdaf44
parent092fa9c14831bc0460bd25190fa013697f572354
[Minor] Dkim_signing: correct is_skip_sign logic

If any of "sign_networks", "auth_only", or "sign_local" are disabled,
then it should not automatically proceed with signing if the enabled
conditions all fail. For example, if only the auth_only setting is
enabled, and is_authed is false, then signing should be skipped.

An earlier check luckily prevents this correctness issue from being
exploitable ("ignoring unauthenticated mail"), but fix the logic anyway.
lualib/lua_dkim_tools.lua