> With that in mind, I made the following changes to qpsmtpd-dev and committed > them.
While that looks good I would propose something completely different: Throw out all the code and replace with this one line: return (DECLINED) if ( $connection->relay_client ); We already have the check_relay plugin which sole purpose is to determine if this is a relay_client. There is no point in duplicating that function in sender_permitted_from or other plugins. We shall only make the relay_client plugin then be able to handle IPv6 entries in the relayclients config. Here I propose to only allow CIDR syntax (aabb:ccdd::/xx) and also reimplement the IPv4 checking so that it can handle CIDR syntax (a.b.c.d/xx) as well as stuff like 'x.y.z' (meaning x.y.z.0/24). Regards, Michael -- It's an insane world, but i'm proud to be a part of it. -- Bill Hicks
