Thanks to visit codestin.com
Credit goes to github.com

Skip to content

MaxMindBinaryProvider, GeoIP and GeoIP2 #302

Closed
@willdurand

Description

@willdurand

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?

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions