John Peacock wrote:
[...]
These three I'm not going to commit (without discussion), because I
don't like the code duplication involved. Rather than patching every
plugin to respect the check_relay() setting (and yes, I know, I have
done this recently), and remember to add that code to any *new* plugin,
I'd rather have a relay_ok() plugin which hooked mail and rcpt and just
shortcircuits the tests, thus making it is site preference to do this
rather than global.
[...]
Sounds good to me. I didn't like patching each of them either. I did
this when testing SMTP AUTH, which can change a non-relay address into a
relayclient.
I'll keep them as local patches for now.
Thanks,
Gordon