On Wed, Feb 21, 2007, Jens Weibler <[EMAIL PROTECTED]> said:

> m. allan noah wrote:
>> i see a block of code in sender_permitted_from's hook_mail that
>> manually parses relayclients and morerelayclients. it checks
>> $self->qp->connection->relay_client() first, so using check_relay
>> before sender_permitted_from would instead seem to be more
>> correct.
>>
>> i wanted to run spf checks on relayclients, so i
>> removed that block of code. greylisting, dnsbl, etc dont have
>> this block, so it begs to be removed from devel version imho...
> There must be a more generic solution. Some people want to use SPF on
> relaying, some like virus-scanning but no spam-checks.
> 
> My wish would be: I can configure inside the plugins-config which
> plugins where used normally and which are used while relaying.

i can see several ways to do this:

1. each plugin gets a 'skip_relayclients' boolean config option
2. a special 'accept_relay' plugin gets loaded repeatedly, once per
   hook, in just the right spot to break before certain plugins
3. have just two plugins in control/plugins, check_relay and 'uber_plugin'
   which will read control/relay_plugins and control/nonrelay_plugins, and
   will rebuild the hook chains based on the relay status.

2 is easiest, 1 and 3 more flexible.

comments?

allan

--
m. allan noah
IT Director, TfHS.net
ph# (804) 355-5489
tf# (866) 724-9722
fx# (804) 355-0477


Reply via email to