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

Reply via email to