fix: reduce error levels to warn only

Not everyone MUST currently use mx or dmarc to send and receive email. e.g. a particular LMS might not utilise mx records at all.
catalyst · Sep 10, 2024 · 34b6460 · 34b6460
1 parent 25871da
commit 34b6460
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/classes/check/dnsdmarc.php b/classes/check/dnsdmarc.php
@@ -75,7 +75,7 @@ public function get_result(): result {
 
         if (empty($dmarc)) {
             $details .= "<p>DMARC record is missing</p>";
-            $status = result::ERROR;
+            $status = result::WARNING;
             $summary = "DMARC DNS record missing";
         } else {
             $details .= "<p>DMARC record found on domain <code>$dmarcdomain</code><br><code>$dmarc</code></p>";

diff --git a/classes/check/dnsmx.php b/classes/check/dnsmx.php
@@ -75,7 +75,7 @@ public function get_result(): result {
 
         if (empty($mxdomains)) {
             $details .= "<p>MX record is missing</p>";
-            $status = result::ERROR;
+            $status = result::WARNING;
             $summary = "MX DNS record missing";
         } else {
             $allmxdomains = join('<br>', array_map(function ($x) {