#3473 closed Bug (fixed)

GeoIP fails silently when mbstring extension not available

Reported by: matt Owned by: capedfuzz
Priority: major Milestone: 1.9.1 - Piwik 1.9.1
Component: Core Keywords:
Cc: Sensitive: no


Reported in forums.

We should

  • Remove silent fail @ operator in all GeoIP drivers
  • Update the System check, to add a warning that, when the extension mbstring is not available or enabled, geoIP will not work
  • It would be nice & important to ensure that, any error happening during GeoIP tracking, would alos be surfaced in the GeoIP tooltips on the admin page, so that users can see the errors in the UI and do not have to debug the tracking requests to see the errrors.

Change History (2)

comment:1 Changed 18 months ago by matt (mattab)

  • Owner set to capedfuzz
  • Priority changed from normal to major

comment:2 Changed 18 months ago by capedfuzz (diosmosis)

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

(In [7290]) Fixes #3473, add "GeoIP won't work" warning to installation system check for mbstring, make sure GeoIP (Php) implementation checks for mbstring plugin, and remove silent fail calls in GeoIP plugins.

Note: See TracTickets for help on using tickets.