6.0.0-beta1
9/24/25

[#4901] missing x-header causes spamassassin hit
Summary missing x-header causes spamassassin hit
Queue IMP
Queue Version 3.2.3
Type Enhancement
State Rejected
Priority 1. Low
Owners
Requester kevin (at) ualberta (dot) net
Created 01/15/2007 (6827 days ago)
Due
Updated 01/19/2007 (6823 days ago)
Assigned
Resolved 01/19/2007 (6823 days ago)
Milestone
Patch No

History
01/19/2007 04:45:59 PM Michael Slusarz Comment #5
State ⇒ Rejected
Reply to this comment
Better yet, just add a X-Mailer header to the conf/header.php (or 
conf/header.txt) file.
01/19/2007 03:06:51 PM vilius (at) lnk (dot) lt Comment #4 Reply to this comment
Upgrade spamassassin. I cannot confirm this under 3.1.4 and can't find 
it on http://spamassassin.apache.org/tests_3_1_x.html, so suppose they 
removed this check.
01/16/2007 05:00:22 PM Jan Schneider Comment #3
State ⇒ Feedback
Reply to this comment
I would say this rule is really broken. I guess it is assuming that 
only MS products (that set the X-Mailer header) are using X-Priority 
too. Did you ask about the rationale behind this rule?
01/15/2007 10:03:06 PM kevin (at) ualberta (dot) net Comment #2 Reply to this comment
2 things...



1.  I removed the other spamassassin hits for clarity.  In this case 
it's -98 because it was a whitelisted user.



2.  I was going to also suggest that if an X-Mailer header was to be 
added it should either be the name given to the server (Horde by 
default), OR just the same value as the User-Agent.
01/15/2007 09:59:21 PM kevin (at) ualberta (dot) net Comment #1
Priority ⇒ 1. Low
Type ⇒ Enhancement
Summary ⇒ missing x-header causes spamassassin hit
Queue ⇒ IMP
State ⇒ New
Reply to this comment
In /imp/compose.php, line 987 you have the following:



   /* Add the X-Priority header, if requested. This appears here since

        this is the "general" location that other mail clients insert

        this header. */

     if ($prefs->getValue('set_priority') &&

         Util::getFormData('x_priority')) {

         $msg_headers->addHeader('X-Priority', 
Util::getFormData('x_priority'));

     }





That's great except if you don't also add an X-Mailer header, 
Spamassassin complains:



X-Spam-Checker-Version: SpamAssassin 2.64 (2004-01-11) on

        mailg.srv.ualberta.ca

X-Spam-Level:

X-Spam-Status: No, hits=-98.8 required=2.0 tests=BAYES_00,PRIORITY_NO_NAME,

        USER_IN_WHITELIST autolearn=no version=2.64

X-Spam-Report:

        *  1.2 PRIORITY_NO_NAME Message has priority setting, but no X-Mailer



I've done a little hack which just adds an X-Mailer header after the 
X-Priority and it solves the problem.  Perhaps it should be there by 
default if IMP is going to add a priority?




Saved Queries