Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add DB() method to return underlying maxminddb #68

Closed
wants to merge 1 commit into from
Closed

Add DB() method to return underlying maxminddb #68

wants to merge 1 commit into from

Commits on Apr 23, 2024

  1. Add DB() method to return underlying maxminddb 

    This ties in to my other patch at [1]; my goal is that geoIP entries are
inserted "on the fly" in the SQL database (which simplifies some
queries) when they're first seen; this removes the need to pre-populate
the database, makes sure the SQL database matches with the GeoIP
database, and future-proofs things better (new countries come in to
existence; for example the list I previously used didn't include
XK/Kosovo).

Right now there is no easy way to get at the maxminddb reader; I'd have
to create my own little struct which wraps the mmdbreader and scans in
the correct types. Not very difficult, but this is a bit easier.

[1]: oschwald/maxminddb-golang#75
    @arp242
    arp242 committed Apr 23, 2024
    Configuration menu
    Copy the full SHA
    e7ea220 View commit details
    Browse the repository at this point in the history