https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6599

--- Comment #11 from Mark Martinec <[email protected]> 2011-11-28 14:53:20 
UTC ---
> Any objections on using Geo::IP as default for 3.4?
> Probably should have used it since few years already.

No objections from me.
The clause can go into the plugin's POD/manpage, I hope this suffices.


> We can leave IP::Country::Fast as backup.

Matt Dainty wrote:
> I think I just scratched this itch as I noticed the RelayCountry plugin
> wasn't doing anything for IPv6 addresses.
> I've put the plugin here:
>   https://github.com/bodgit/spamassassin-relaycountry-geoip
> I've named it RelayCountry2 so it can live alongside the current plugin but
> will update the same metadata so it's a case of enable one or the other.
> I've replicated the behaviour of IP::Country in marking private addresses
> with "**" and I've added similar rules for IPv6 based on the equivalent RFC.

I wouldn't like to see two variants of functionally equivalent plugins
and letting people scratch their heads, deciding which of them to use.
Either switch entirely to Geo::IP and ditch IP::Country::Fast, or
merge both into a single plugin and let it fallback to IP::Country::Fast
if Geo::IP is not installed.

-- 
Configure bugmail: 
https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

Reply via email to