mainSpamAssassin Milter Plugin - Support: sr #103491, Feature Request: configurable...

 
 

sr #103491: Feature Request: configurable block message

Submitter:  None
Submitted:  Mon 18 Oct 2004 02:11:19 PM UTC
Votes: 49
 
Category:  None Priority:  5 - Normal
Severity:  1 - Wish Status:  Duplicate
Privacy:  Public Assigned to:  None
Originator Email:  -email is unavailable- Open/Closed:  Closed
Operating System:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Fri 15 Aug 2014 02:27:22 AM UTC, comment #5: 

a variation on this was applied to cvs and will bein the next release.  (the freebsd patch).

Todd Kover <kovert>
Group administrator
Fri 13 Nov 2009 09:18:23 PM UTC, comment #4: 

I suggest that this might be even more useful if the reject message contained the queue id of the message.  In case of a false positive, that would make it much easier to grep for why it was rejected.

David Burns <davburns>
Wed 24 Sep 2008 10:06:23 PM UTC, comment #3: 

Duh. I can't believe this patch has not been integrated. it's over 4 years old, and a much-needed feature. Bad maintainership!

I just implemented this myself, not knowing that there is already a patch for it. It can take the string from either env var (added after being inspired by the patch here) or from a commandline arg. Only problem is that I don't know how to pass long strings with spaces so that getopt will accept them as one option - usually, that's "", but that didn't work.

Ben Bucksch <benb>
Wed 10 Aug 2005 11:17:03 PM UTC, comment #2: 

Changelog for version custom-reject-3.patch:

  • Version custom-reject-2.patch produced a segfault when no env

  variable was found. This is fixed in the new version.
  Thanks to John Hill for reporting this bug to me!

  • spamass-milter now writes a short note to the logs when it

  could not find the environment variable.
  ("REJECT_MESSAGE not set; using default" or
   "reject message (env) not set; using default")

On Debian systems simply add a line like
REJECT_MESSAGE="my custom reject with url"
to /etc/default/spamass-milter and it will be used. You might as well use a special variable and specify it with -C.

On other distributions there may be other mechanisms. If everything else fails: simply add a line like:
export REJECT_MESSAGE="my custom reject with url"
in spamass-milters' startup item.

Adi Kriegisch (adi@vrvis.at)

Anonymous
Mon 18 Oct 2004 03:38:41 PM UTC, comment #1: 

I cleaned up my patch (ie removed the nasty ADI comments) and added a "-C" option to allow the user/admin to specify an environment variable containing the string to use as reject comment...
And, most important: credits go to Gerd Hesina for helping me with this!! Thank you very much!

Anonymous
Mon 18 Oct 2004 02:11:19 PM UTC, original submission:  

As I need a configurable reject message (I want to redirect false positives to a URL), I made the attached changes to spamass-milter (see patchfile).
simply do

export REJECT_MESSAGE="No spam here! (see http://mycompany.com/spam for details)"

to give those who were (erroneously) caught by my spamfilter a way how to reach me.
Then you only have to start spamass-milter with the newly introduced "-c" option...

best regards,
  Adi Kriegisch (adi@vrvis.at)

PS: I thought about using a config file for this but I decided that this does not fit to the concept of spamass-milter as it is now.
PPS: maybe its a good idea to do a "-C" option where one can specify the name of the environment variable as well... (so noone is forced to use REJECT_MESSAGE).

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #16566:  reject-message.4.diff added by benb (2KiB - text/x-diff - Commandline option or env var)
file #4018:  custom-reject-3.patch added by None (3KiB - text/x-diff - custom-reject-3.patch)
file #3898:  custom-reject-2.patch added by None (2KiB - text/x-diff)
file #3897:  custom-reject.patch added by None (2KiB - text/x-diff)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by kovert (Posted a comment)
  • -email is unavailable- added by davburns (Voted in favor of this item)
  • -email is unavailable- added by davburns (Posted a comment)
  • -email is unavailable- added by benb (Posted a comment)
  •  

    There are 49 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 7 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2014-08-15 kovert StatusNone Duplicate
        Open/ClosedOpen Closed
    2009-11-13 davburns Carbon-Copy- Added davburns
    2008-09-24 benb Attached File- Added reject-message.4.diff, #16566
    2005-08-10 None Attached File- Added custom-reject-3.patch, #151
    2004-10-18 None Attached File- Added custom-reject-2.patch, #77
    2004-10-18 None Attached File- Added custom-reject.patch, #76

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code