Check DMARC sub/domain policy only

1 year ago · d81a61a3f6
parent 0197d736ff
commit d81a61a3f6
1 changed files with 4 additions and 1 deletions
--- a/app/src/main/java/eu/faircode/email/MessageHelper.java
+++ b/app/src/main/java/eu/faircode/email/MessageHelper.java
@ -2176,7 +2176,10 @@ public class MessageHelper {
                            Boolean none = null;
                            for (String p : v.substring(s + 1, e).split("\\s+")) {
                                String[] kv = p.split("=");
-                                if (kv.length == 2 && "none".equalsIgnoreCase(kv[1]))
+                                // Without getting the DMARC DNS record, it isn't possible to check the sub/domain
+                                if (kv.length == 2 &&
+                                        ("p".equalsIgnoreCase(kv[0]) || "sp".equalsIgnoreCase(kv[0])) &&
+                                        "none".equalsIgnoreCase(kv[1]))
                                    none = (none == null || none) && "none".equalsIgnoreCase(kv[1]);
                            }
                            if (none != null && none)