Browse Source

[Minor] Change the default list of oversigned headers

Trivia:

`Subject` header needs to be oversigned as an attacker could add some
'bad' subject to DKIM signed emails with no subject (rare but possible
case). This header is clearly displayed to a user hence, its presence as
well as absence MUST be oversigned explicitly.

`Reply-To` header is widely used to designate a special address used for
replies only but not for authentication checks. It is thus possible to
add a malicious `Reply-To` header to force users to reply to a DKIM
signed email to some attacker's controlled email address. It clearly
opens surface for social engineering like attacks and this header must
thus be unconditionally oversigned even if not presented in an original
message.

Issue: #2887
tags/1.9.3
Vsevolod Stakhov 5 years ago
parent
commit
23a41dae4d
1 changed files with 2 additions and 2 deletions
  1. 2
    2
      src/plugins/dkim_check.c

+ 2
- 2
src/plugins/dkim_check.c View File

@@ -54,13 +54,13 @@
static const gchar *M = "rspamd dkim plugin";

static const gchar default_sign_headers[] = ""
"(o)from:(x)sender:(x)reply-to:(x)subject:(x)date:(x)message-id:"
"(o)from:(x)sender:(o)reply-to:(o)subject:(x)date:(x)message-id:"
"(o)to:(o)cc:(x)mime-version:(x)content-type:(x)content-transfer-encoding:"
"resent-to:resent-cc:resent-from:resent-sender:resent-message-id:"
"(x)in-reply-to:(x)references:list-id:list-help:list-owner:list-unsubscribe:"
"list-subscribe:list-post:(x)openpgp:(x)autocrypt";
static const gchar default_arc_sign_headers[] = ""
"(o)from:(x)sender:(x)reply-to:(x)subject:(x)date:(x)message-id:"
"(o)from:(x)sender:(o)reply-to:(o)subject:(x)date:(x)message-id:"
"(o)to:(o)cc:(x)mime-version:(x)content-type:(x)content-transfer-encoding:"
"resent-to:resent-cc:resent-from:resent-sender:resent-message-id:"
"(x)in-reply-to:(x)references:list-id:list-help:list-owner:list-unsubscribe:"

Loading…
Cancel
Save