@bobfrankston/rmfmail 1.2.284 → 1.2.286

package/client/app.bundle.js

@@ -3109,19 +3109,22 @@ async function showMessage(accountId, uid, folderId, specialUse, isRetry = false
     bodyEl.innerHTML = "";
     const spam = msg.spamScore;
     if (spam && Number.isFinite(spam.score) && (spam.flagged || spam.score >= spam.threshold / 2)) {
-      const level = spam.flagged ? "high" : "mid";
+      const benign = !!spam.proved || spam.kind === "bulk";
+      const level = benign ? "bulk" : spam.flagged ? "high" : "mid";
       const chip = document.createElement("div");
       chip.className = `mv-spamscore mv-spamscore-${level}`;
-      chip.title = level === "high" ? "Your mail server scored this at or above its spam threshold and flagged it." : "Below your server's spam threshold, but more than halfway to it.";
+      const why = (spam.reasons || []).length ? "\n\n" + spam.reasons.join("\n") : "";
+      chip.title = (level === "bulk" ? spam.proved ? "Scored by bulk-mail or blocklist rules; the sending domain passed DMARC, so the sender is proved." : "Every rule that scored is a bulk-mail test \u2014 none of them test who sent this." : level === "high" ? "Your mail server scored this at or above its spam threshold and flagged it." : "Below your server's spam threshold, but more than halfway to it.") + why;
       chip.textContent = `spam score ${spam.score} of ${spam.threshold}`;
       bodyEl.appendChild(chip);
     }
     const trust = msg.trust || [];
     if (trust.length) {
-      const worst = trust.some((f) => f.severity === "danger") ? "danger" : "caution";
+      const worst = trust.some((f) => f.severity === "danger") ? "danger" : trust.some((f) => f.severity === "caution") ? "caution" : "info";
+      const head = worst === "danger" ? "This message is not what it says it is" : worst === "caution" ? "Check this message before acting on it" : "";
       const warn = document.createElement("div");
       warn.className = `mv-trust mv-trust-${worst}`;
-      warn.innerHTML = `<div class="mv-trust-head">${worst === "danger" ? "This message is not what it says it is" : "Check this message before acting on it"}</div>` + trust.map((f) => `<div class="mv-trust-item mv-trust-${escapeText(f.severity)}"><div class="mv-trust-text">${escapeText(f.text)}</div><div class="mv-trust-detail">${escapeText(f.detail)}</div></div>`).join("") + `<div class="mv-trust-foot">Links and attachments here should not be opened.</div>`;
+      warn.innerHTML = (head ? `<div class="mv-trust-head">${head}</div>` : "") + trust.map((f) => `<div class="mv-trust-item mv-trust-${escapeText(f.severity)}"><div class="mv-trust-text">${escapeText(f.text)}</div><div class="mv-trust-detail">${escapeText(f.detail)}</div></div>`).join("") + (worst === "info" ? "" : `<div class="mv-trust-foot">Links and attachments here should not be opened.</div>`);
       bodyEl.appendChild(warn);
     }
     if (msg.hasRemoteContent) {