[xoops-trackers] [ xoops-Bugs-3571362 ] xoops->getBaseDomai() doesn't parse IPv6 correctly

[SourceForge.net <no...@so...>]

Bugs item #3571362, was opened at 2012-09-24 17:08
Message generated for change (Tracker Item Submitted) made by zyspec
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=3571362&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core -  Core
Group: XOOPS 2.6.x
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: zyspec (zyspec)
Assigned to: Nobody/Anonymous (nobody)
Summary: xoops->getBaseDomai() doesn't parse IPv6 correctly

Initial Comment:
In the Xoops class (./class/xoops.php) there is a method called getBaseDomain that does not correctly parse IP addresses.  Specifically it will not parse ipv6 correctly.  I believe the following code:

 // first check for ip address
if (count($DOMAIN) == 4 && is_numeric($DOMAIN[0]) && is_numeric($DOMAIN[3])) {
    return $full_domain;
}

should be changed to:

 // first check for either ip4 or ip6 address
if (filter_var($full_domain, FILTER_VALIDATE_IP)) {
    return $full_domain;
}


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=3571362&group_id=41586