The URL that appears in the Status Report (../admin/reports/status) to download the Geolite database is broken. It should be http://geolite.maxmind.com/download/geoip/database/GeoLiteCountry/GeoIP....

CommentFileSizeAuthor
#5 geoip_948290.patch1.15 KBdrewish
#4 geopip_948290.patch1.11 KBdrewish

Comments

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented

Actually it is not only in the site's 'Status Report' page, but also the module's settings page (../admin/settings/geoip).

This shouldn't be a static URL to a single file, since people have the option to either use the simple country database or the city database file. If there's a way to detect which of the two is used, then the perfect solution would be to point to the correct version.

drewish’s picture

drewish commented

I guess we can just provide both links. Does http://geolite.maxmind.com/download/geoip/database/GeoLiteCity.dat.gz seem right for the cities?

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented

yep! that link is correct.

drewish’s picture

drewish commented
Status: Active » Needs review
StatusFileSize
new geopip_948290.patch1.11 KB

How's this look?

drewish’s picture

drewish commented
StatusFileSize
new geoip_948290.patch1.15 KB

Last patch had a bad URL. This one also passes the MaxMind.com URL in via translation token. There have been a few issues where people seem confused that the city database has a different name and you can adjust the path rather than renaming the file so I tried to clarify the instructions.

drewish’s picture

drewish commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.