Opened on 06/16/2014 at 01:59:09 PM

Closed on 06/23/2014 at 10:32:14 AM

#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.

Attachments (0)

Change History (3)

comment:1 Changed on 06/17/2014 at 03:13:55 PM 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 on 06/17/2014 at 03:21:57 PM 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 on 06/23/2014 at 10:32:14 AM by trev

  • Resolution set to fixed
  • Status changed from reviewing to closed

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from (none).
 
Note: See TracTickets for help on using tickets.