Update SpamTestSPF.cpp

SFP Explanation is almost never returned, and I got somewhat irritated by the 'empty' parentheses in the error message
mpkleine · Apr 17, 2022 · 2fbd163 · 2fbd163
1 parent 0798e1e
commit 2fbd163
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/hmailserver/source/Server/Common/AntiSpam/SpamTestSPF.cpp b/hmailserver/source/Server/Common/AntiSpam/SpamTestSPF.cpp
@@ -59,7 +59,10 @@ namespace HM
       if (result == SPF::Fail)
       {
          // Blocked by SPF.s
-         sMessage.Format(_T("Blocked by SPF (%s)"), sExplanation.c_str());
+         if (!sExplanation.IsEmpty())
+            sMessage.Format(_T("Blocked by SPF. (%s)"), sExplanation.c_str());
+         else
+            sMessage = "Blocked by SPF.";
          iScore = Configuration::Instance()->GetAntiSpamConfiguration().GetUseSPFScore();
 
          std::shared_ptr<SpamTestResult> pResult = std::shared_ptr<SpamTestResult>(new SpamTestResult(GetName(), SpamTestResult::Fail, iScore, sMessage));