Bug 3718 - Mail::SpamAssassin::Conf - report_safe, remove_header all Report
Summary: Mail::SpamAssassin::Conf - report_safe, remove_header all Report
Status: RESOLVED INVALID
Alias: None
Product: Spamassassin
Classification: Unclassified
Component: Documentation (show other bugs)
Version: SVN Trunk (Latest Devel Version)
Hardware: Other other
: P5 normal
Target Milestone: Future
Assignee: SpamAssassin Developer Mailing List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-08-23 02:05 UTC by Jesus Climent
Modified: 2021-04-18 10:54 UTC (History)
1 user (show)


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 Jesus Climent 2004-08-23 02:05:53 UTC 
in Manual Mail::SpamAssassin::Conf it is not entirely clear that simply
adding this line will now work:

       always_add_report { 0 | 1 }   (default: 0)
           ...
           option instead:

           add_header all Report _REPORT_

Please clarify this by stating that it is mandatory to set
report_safe before touching "Report" setting. Like this:

        report_safe             0
        remove_header all Report

This will not work:

        remove_header all Report
        report_safe             0
Comment 1 Theo Van Dinter 2004-08-23 09:09:57 UTC 
Subject: Re:  New: Mail::SpamAssassin::Conf - report_safe, remove_header all Report

On Mon, Aug 23, 2004 at 02:05:54AM -0700, bugzilla-daemon@bugzilla.spamassassin.org wrote:
> in Manual Mail::SpamAssassin::Conf it is not entirely clear that simply
> adding this line will now work:
> 
>        always_add_report { 0 | 1 }   (default: 0)
>            ...
>            option instead:
> 
>            add_header all Report _REPORT_

??  always_add_report doesn't exist in 3.0.  it's not in the Conf POD at all.

> Please clarify this by stating that it is mandatory to set
> report_safe before touching "Report" setting. Like this:
>         report_safe             0
>         remove_header all Report
> This will not work:
>         remove_header all Report
>         report_safe             0

So the current documentation isn't clear?:

If [report_safe] is set to 0, incoming spam is only modified by adding
some C<X-Spam-> headers and no changes will be made to the body.  In
addition, a header named B<X-Spam-Report> will be added to spam.  You
can use the B<remove_header> option to remove that header after setting
B<report_safe> to 0.
Comment 2 Daniel Quinlan 2005-03-30 01:08:27 UTC 
move bug to Future milestone (previously set to Future -- I hope)
Comment 3 Henrik Krohns 2021-04-18 10:54:41 UTC 
Closing old stale bug, I don't think there's anything to fix here.