search

msimerson / Mail-Toaster-6

Mail Toaster 6
https://github.com/msimerson/Mail-Toaster-6/wiki
BSD 3-Clause "New" or "Revised" License
46 stars 16 forks source link

geoip.sh fails using maxmind-geolite-mirror while downloading DB's #591

Open rricker1 opened 15 hours ago

rricker1 commented 15 hours ago

Describe the bug When attempting to download DB's maxmind-geolite-mirror stalls while maxing out connection attempts before finally failing. I receive an email notification "Daily GeoIP Database Download Limit Reached".

To Reproduce provision geoip

Expected behavior databases should be downloaded, jail should be promoted.

Server (please complete the following information): FreeBSD-13.3

Additional context Adding: export GEOIP_UPDATER="geoipupdate" export MAXMIND_ACCOUNT_ID="XXXXXXX"

to mail mail-toaster.conf completes as expected.

Q. Is maxmind-geolite-mirror preferfred over geoipupdate?

msimerson commented 14 hours ago

A: maxmind-geolite-mirror is currently broken. See msimerson/maxmind-geolite-mirror#47

rricker1 commented 13 hours ago

I'm not sure that my question was answered. Is one better/preferred than the other?

Should export GEOIP_UPDATER="" and export MAXMIND_ACCOUNT_ID="" be included in mail-toaster.conf.

msimerson commented 13 hours ago

Yes, the one that works is preferred.

They're both very simple utilities that fetch files into a local directory. If MaxMind hasn't made a mess with their old perl based updater, I'd have never bothered to write maxmind-geolite-mirror. Now that they've got a well functioning updater, and they're probably going to keep futzing with how they're hosting the files, it's probably best to deprecate maxmind-geolite-updater and prefer geoipupdate.

rricker1 commented 13 hours ago

Thank you for the explanation.