Merge pull request #270 from nccgroup/issue_258

x4v13r64 · web-flow · commit 0741765076f2 · 2018-07-03T11:48:52.000+02:00
Fix for issue #258.
diff --git a/AWSScout2/rules/data/findings/ses-identity-dkim-not-enabled.json b/AWSScout2/rules/data/findings/ses-identity-dkim-not-enabled.json
@@ -1,6 +1,6 @@
 {
     "description": "DKIM not enabled",
-    "rationale": "",
+    "rationale": "DKIM signing is not enabled for emails sent from the identity.",
     "path": "ses.regions.id.identities.id",
     "dashboard_name": "Identities",
     "conditions": [ "and",
diff --git a/AWSScout2/rules/data/findings/ses-identity-dkim-not-verified.json b/AWSScout2/rules/data/findings/ses-identity-dkim-not-verified.json
@@ -1,6 +1,6 @@
 {
-    "description": "DKIM not enabled",
-    "rationale": "",
+    "description": "DKIM not verified",
+    "rationale": "Amazon SES has not verified the DKIM DNS records (tokens) published in the domain name's DNS.",
     "path": "ses.regions.id.identities.id",
     "dashboard_name": "Identities",
     "conditions": [ "and",

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"description": "DKIM not enabled",`
`3`		`- "rationale": "",`
	`3`	`+ "rationale": "DKIM signing is not enabled for emails sent from the identity.",`
`4`	`4`	`"path": "ses.regions.id.identities.id",`
`5`	`5`	`"dashboard_name": "Identities",`
`6`	`6`	`"conditions": [ "and",`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`		`- "description": "DKIM not enabled",`
`3`		`- "rationale": "",`
	`2`	`+ "description": "DKIM not verified",`
	`3`	`+ "rationale": "Amazon SES has not verified the DKIM DNS records (tokens) published in the domain name's DNS.",`
`4`	`4`	`"path": "ses.regions.id.identities.id",`
`5`	`5`	`"dashboard_name": "Identities",`
`6`	`6`	`"conditions": [ "and",`