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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

klonos’s picture

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

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

yep! that link is correct.

drewish’s picture

Status: Active » Needs review
FileSize
1.11 KB

How's this look?

drewish’s picture

FileSize
1.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

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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