https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7449
--- Comment #2 from Karsten Bräckelmann <[email protected]> --- Thanks Deepen for your contribution, much appreciated. I've had a look at your SA plugin, so here's some quick first comments: While your documentation is perfectly fine advising to add the configuration to the SA local.cf file, it generally makes sense to keep custom plugin configuration in its own file. That is, just rename add_to_local.cf for example promotional.cf and keep that file in the same directory as local.cf. Also, loadplugin directives should better be placed in .pre files, which are guaranteed to be processed before any .cf files. (Since you keep all your custom plugin configuration after the loadplugin line, this also works in cf files.) The most important advice is this: Your check_for_promotions() eval rule is called for each message, and should never read custom files for configuration. Any per-plugin configuration should be read once only at initialization. Also it is better to use custom SA configuration instead of some ini file. Looking at your ini file it appears instead of the custom plugin, you could have accomplished the task using a couple non-scoring double underscore sub-rules and a meta rule combining them. The sieve script then should check the X-Spam-Status header for that meta rule's name to identify matching mail. Regarding editing the wiki: You can do that, but first need to send a brief request to the dev@ list so we can grant edit permissions (see the wiki's frontpage). We've had to restrict self-signup to prevent wiki spam and vandalism, unfortunately. For now though, my advise would be to first polish up the custom plugin before posting / linking it from the SA wiki. -- You are receiving this mail because: You are the assignee for the bug.
