When RequiredFrom rejects a message, reply with the reason to do so.

ghen2 · ghen2 · commit 7b41e66aa594 · 2022-02-06T18:22:08.000+01:00
(equivalent to trusteddomainproject#203 for RequiredHeaders)
diff --git a/opendmarc/opendmarc.c b/opendmarc/opendmarc.c
@@ -2163,7 +2163,11 @@ mlfi_eom(SMFICTX *ctx)
 		}
 
 		if (conf->conf_reqfrom)
+		{
+			status = dmarcf_setreply(ctx, DMARC_REJECT_SMTP, DMARC_REJECT_ESC,
+				"RFC5322 requirement error: missing From field");
 			return SMFIS_REJECT;
+		}
 		else
 			return SMFIS_ACCEPT;
 	}

Original file line number	Diff line number	Diff line change
`@@ -2163,7 +2163,11 @@ mlfi_eom(SMFICTX *ctx)`
`2163`	`2163`	`}`
`2164`	`2164`
`2165`	`2165`	`if (conf->conf_reqfrom)`
	`2166`	`+ {`
	`2167`	`+ status = dmarcf_setreply(ctx, DMARC_REJECT_SMTP, DMARC_REJECT_ESC,`
	`2168`	`+ "RFC5322 requirement error: missing From field");`
`2166`	`2169`	`return SMFIS_REJECT;`
	`2170`	`+ }`
`2167`	`2171`	`else`
`2168`	`2172`	`return SMFIS_ACCEPT;`
`2169`	`2173`	`}`