https://bz.apache.org/SpamAssassin/show_bug.cgi?id=8331

--- Comment #7 from Giovanni Bechis <giova...@paclan.it> ---
Few comments to the code:
- private subs should start with a "_", 'sub connect_read' should become 'sub
_connect_read'
- regexps that uses variables controlled by the user like '$msgid =~
s/^$self->{key_prefix}s://;', must be "compiled" using 'compile_regexp' sub
from Mail::SpamAssassin::Util.
An example of this code is in Mail::SpamAssassin::Plugin::OLEVBMacro

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to