Closed
Description
As mentioned in #301, the current implementation of MaxMindBinaryProvider
is far from perfect. This is mostly because of the geoip/geoip
package we use under the hood.
Obviously there is a newer lib, named geoip2/geoip2
, which would be better. I'd like to switch to this package, but it would imply a BC break, so I suggest the introduction of a new provider (GeoIP2Provider
? MaxMindBinary2Provider
?) leveraging the geoip2/geoip2
lib. Also, we should mark MaxMindBinaryProvider
as deprecated.
WDYT?