https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7529
--- Comment #26 from Kent Oyer <[email protected]> --- > > You can use a config option as I have done in > https://bz.apache.org/SpamAssassin/attachment.cgi?id=5523&action=view > > Anyway IMHO the "problems" with using GeoIP2 api are: > - GeoIP2 is not packaged in many operating systems > - GeoIP2 xs code is 32bit only In my patch, if GeoIP2 is not installed I fall back to the legacy Geo::IP first and then to IP::Country::Fast. This method does not break existing installations but still allows people to use GeoIP2 if desired. Why not do that? Thanks for the tip on the config option. I thought about that but by moving the code into extract_metadata we have to instantiate a new Reader object with every message. I guess it's a tradeoff. -- You are receiving this mail because: You are the assignee for the bug.
