-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Closed
Labels
BugFor errors / faults / flaws / inconsistencies etc.For errors / faults / flaws / inconsistencies etc.Help wantedBeginner friendly issues or issues where we'd highly appreciate community's help and involvement.Beginner friendly issues or issues where we'd highly appreciate community's help and involvement.
Milestone
Description
Another beginner friendly bug:
When I click on the ISP Rostelecom
it wan't to go to http://www.rostelecom/
as it thinks the providername is an URL ending with com
. As many ISPs are called telecom I guess this could happen quite often:
matomo/plugins/Provider/Provider.php
Lines 82 to 84 in 9243b9a
$extToExclude = array( | |
'com', 'net', 'org', 'co' | |
); |
matomo/plugins/Provider/Provider.php
Lines 118 to 126 in 9243b9a
$e = explode('.', $hostname); | |
$s = sizeof($e); | |
// if extension not correct | |
if (isset($e[$s - 2]) && in_array($e[$s - 2], $extToExclude)) { | |
return $e[$s - 3] . "." . $e[$s - 2] . "." . $e[$s - 1]; | |
} else { | |
return $e[$s - 2] . "." . $e[$s - 1]; | |
} |
PS: This code was added in def7586 11 years ago!
Metadata
Metadata
Assignees
Labels
BugFor errors / faults / flaws / inconsistencies etc.For errors / faults / flaws / inconsistencies etc.Help wantedBeginner friendly issues or issues where we'd highly appreciate community's help and involvement.Beginner friendly issues or issues where we'd highly appreciate community's help and involvement.