Closed (fixed)
Project:
Whois lookup
Version:
6.x-1.1
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
4 Sep 2008 at 08:27 UTC
Updated:
25 Dec 2009 at 09:30 UTC
Jump to comment: Most recent file
Comments
Comment #1
Gurpartap Singh commentedWhoops, I can confirm that this will happen with the url strip-down regexp I'm using in the module. This is actually critical.
Comment #2
sersim commentedI've commented out the regexp in the file whois.module, and it works now.
function whois_parse_url($url) {
//$r = "^(?:(?P\w+)://)?";
//$r .= "(?:(?P\w+):(?P
\w+)@)?";
//$r .= "(?P(?:(?P[\w\.]+)\.)?" . "(?P\w+\.(?P\w+)))";
//$r .= "(?::(?P
\d+))?";
//$r .= "(?P
[\w/]*/(?P\w+(?:\.\w+)?)?)?";
//$r .= "(?:\?(?P[\w=&]+))?";
//$r .= "(?:#(?P\w+))?";
//$r = "!$r!"; // Delimiters
//preg_match($r, $url, $result);
//return $result[6];
return $url;
}
Comment #3
Gurpartap Singh commentedComment #4
Gurpartap Singh commentedMarked http://drupal.org/node/333050 as a duplicate.
Comment #5
helmo commentedThe attached patch expands the regexp to include some multi-level tld's. However this currently requires a list of all these cases, which is hard to maintain.
Comment #6
helmo commentedFix for #653156: Unfixable whois_parse_url, which should resolve this issue, has been committed.
Please re-open if the problem persists.