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.

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

CSV support would be key to be able to use this in conjunction with geolite2legacy #272

Open
rcfa opened this issue Dec 19, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@rcfa
Copy link

rcfa commented Dec 19, 2023

CSV support would be key to be able to use this in conjunction with https://github.com/sherpya/geolite2legacy to keep some legacy software operating, in particular scripts using geoiplookup, which depends on the legacy format database.

The debian geoipupdate package keeps the /var/lib/GeoIP/ up to date, but /var/share/GeoIP obviously not. All that's needed is a cron script running geolite2legacy. But that requires that geoipupdate can handle CSV files.

@oschwald oschwald added the enhancement New feature or request label Jan 8, 2024
@oschwald
Copy link
Member

oschwald commented Jan 8, 2024

I don't know that we are likely to add CSV support to this tool. The MMDB files are generally used directly and can easily be updated atomically.

With the CSVs, they are generally ingested as part of a pipeline and they are generally just an intermediate format that does not need to persist on disk. In most cases, it makes more sense to download the files as needed as part of the build process. Also, since most of the CSV databases consist of multiple files, it is difficult to have any guarantees around the updates being atomic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

No branches or pull requests

2 participants