Bug 1361 - report_header No longer works
Summary: report_header No longer works
Status: RESOLVED FIXED
Alias: None
Product: Spamassassin
Classification: Unclassified
Component: spamassassin (show other bugs)
Version: SVN Trunk (Latest Devel Version)
Hardware: Other other
: P2 normal
Target Milestone: 2.50
Assignee: SpamAssassin Developer Mailing List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-01-10 14:25 UTC by Rod Begbie
Modified: 2003-01-10 05:51 UTC (History)
0 users


Attachment Type Modified Status Actions Submitter/CLA Status
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Rod Begbie 2003-01-10 14:25:51 UTC 
The code to find "report_header" was removed from Conf.pm
(http://cvs.sourceforge.net/cgi-bin/viewcvs.cgi/spamassassin/spamassassin/lib/Mail/SpamAssassin/Conf.pm.diff?r1=1.136&r2=1.137)

However, PerMsgStatus still looks for $self->{conf}->{report_header} before it
adds a X-Spam-Report header.

Either report_safe=0 should add X-Spam_report, or (preferably) report_header
should be reinstated to Conf.pm
Comment 1 Theo Van Dinter 2003-01-10 14:47:27 UTC 
Subject: Re: [SAdev]  New: report_header No longer works

On Fri, Jan 10, 2003 at 02:25:52PM -0800, bugzilla-daemon@hughes-family.org wrote:
> However, PerMsgStatus still looks for $self->{conf}->{report_header} before it
> adds a X-Spam-Report header.

Hmmm.

> Either report_safe=0 should add X-Spam_report, or (preferably) report_header
> should be reinstated to Conf.pm

Well, there is no other option (ie: no report in body), so I'll remove
the if statement and call it good. :)
Comment 2 Theo Van Dinter 2003-01-10 14:51:51 UTC 
committed to cvs