Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] iconv() notice in PHP GeoIp2 LocationProvider during fuzzy matching #22321

Open
4 tasks done
diosmosis opened this issue Jun 18, 2024 · 0 comments
Open
4 tasks done
Labels
Potential Bug Something that might be a bug, but needs validation and confirmation it can be reproduced. triaged

Comments

@diosmosis
Copy link
Member

What happened?

A Matomo for WordPress user reports seeing these notices in their PHP error log:

iconv(): Detected an illegal character in input string in /var/www/example.com/htdocs/wp-content/plugins/matomo/app/plugins/GeoIp2/LocationProvider/GeoIp2/Php.php on line 296;

(source: https://wordpress.org/support/topic/php-errors-in-server-log/)

What should happen?

Notices should not be emitted by the PHP GeoIP location provider. The fuzzyMatch method should perhaps ignore characters that cannot be transliterated or errors should be silenced.

How can this be reproduced?

Unknown, though the user in question is using the default DB-IP databases, and geolocation appears to be working (there is data in the Locations reports).

Matomo version

5.1.0

PHP version

No response

Server operating system

No response

What browsers are you seeing the problem on?

No response

Computer operating system

No response

Relevant log output

No response

Validations

@diosmosis diosmosis added Potential Bug Something that might be a bug, but needs validation and confirmation it can be reproduced. To Triage An issue awaiting triage by a Matomo core team member labels Jun 18, 2024
diosmosis added a commit to matomo-org/matomo-for-wordpress that referenced this issue Jul 3, 2024
@randy-innocraft randy-innocraft added triaged and removed To Triage An issue awaiting triage by a Matomo core team member labels Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Potential Bug Something that might be a bug, but needs validation and confirmation it can be reproduced. triaged
Projects
None yet
Development

No branches or pull requests

2 participants