Robert Spier wrote:
What's the goal of this?  It also needs some documentation or examples.

The idea is to allow plugins to do something like:

for my $rcpt ($transaction->recipients) {
    do_stuff() if $spam_score > $rcpt->config('spam_threshold');
}

IOW, per-user prefs. This is the first, obvious requisite to this sort of handling.

I documented config() in the POD for Qpsmtpd::Address itself; I can add some documentation and examples to README.plugins as well, is there any other location that could use documentation?

Also, I noticed some more deficiencies in this patch today, namely that Qpsmtpd::config() needs to pass the Qpsmtpd::Address object to run_hooks('rcpt_config'). Although I did some basic testing to make sure it was functional, I suggest holding off on application until the code has seen more real-world use from me and I submit a follow-up.

Thanks!

-Jared

Reply via email to