Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#19072 closed Bug (fixed)

GeoDjango/GeoIP docs have outdated info

Reported by: Flavio Curella Owned by: nobody
Component: GIS Version: 1.4
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: yes UI/UX: no

Description

The doc about GeoIP states:

[...] These datasets may be
downloaded from MaxMind. Grab the GeoIP.dat.gz and GeoLiteCity.dat.gz
and unzip them in a directory [...]

On the GeoMind server there isn't any file called GeoIP.dat.gz. The file is now called GeoIPv6.dat.gz

Attachments (1)

geoip_doc.diff (1.1 KB) - added by Flavio Curella 4 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 4 years ago by Flavio Curella

Has patch: set

comment:2 Changed 4 years ago by Claude Paroz

Component: DocumentationGIS
Patch needs improvement: set
Triage Stage: UnreviewedAccepted

I think that Django does not support IPv6 data yet (see #18349). The IPv4 file is now located in the GeoLiteCountry directory.

Changed 4 years ago by Flavio Curella

Attachment: geoip_doc.diff added

comment:3 Changed 4 years ago by Flavio Curella

Thanks claudep. Path updated.

comment:4 Changed 4 years ago by Claude Paroz

In your formulation, I'm not sure it's clear enough that one file is in GeoLiteCountry dir and not the other.
What about simply writing the path like that: GeoLiteCountry/GeoIP.dat.gz ? No need for another patch, I'll fix it myself if you agree.

comment:5 Changed 4 years ago by Flavio Curella

I agree with you, let's go with your version.

Thanks,
Flavio.

comment:6 Changed 4 years ago by Claude Paroz <claude@…>

Resolution: fixed
Status: newclosed

In ab8c9703683856eef0346b661e36d6f05db67435:

Fixed #19072 -- Corrected an external file path in GeoIP docs

Thanks Flavio Curella for the report and the initial patch.

comment:7 Changed 4 years ago by Claude Paroz <claude@…>

In 0636c9583fda5a81a18b47876bcddac62bd2e547:

Fixed #19072 -- Corrected an external file path in GeoIP docs

Thanks Flavio Curella for the report and the initial patch.

Note: See TracTickets for help on using tickets.
Back to Top