Opened 5 years ago

Closed 5 years ago

#670 closed change (fixed)

Use GeoIP City database for AWStats

Reported by: trev Assignee:
Priority: P2 Milestone:
Module: Infrastructure Keywords:
Cc: jobp, ben Blocked By:
Blocking: Platform:
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):

http://codereview.adblockplus.org/5316783485485056

Description

Background

Currently our AWStats install only resolves countries, that's not as useful as it could be.

What to change

It seems that GeoIP City database can be used with AWStats (via a plugin?), we should do that.

Change History (3)

comment:1 Changed 5 years ago by trev

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

Note that this didn't work out of the box for me. geoip_city_maxmind.pm module is buggy, function SectionProcessIp_geoip_city_maxmind uses $city instead of $region->city in an if block condition (and it also comments out use strict at the top, great way of doing it). With this issue fixed everything seems to work fine however.

comment:2 Changed 5 years ago by trev

The issue here is http://sourceforge.net/p/awstats/bugs/873/ - which is not fixed in AWStats 7.3 despite the claim there.

comment:3 Changed 5 years ago by trev

  • Resolution set to fixed
  • Status changed from reviewing to closed
Note: See TracTickets for help on using tickets.