Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.0.6
Bug #17564 regex too strict for authoritative lookups
Submitted: 2010-07-06 15:53 UTC
From: hm2k Assigned: kguest
Status: Closed Package: Net_Whois (version SVN)
PHP Version: Irrelevant OS:
Roadmaps: 1.0.4    
Subscription  


 [2010-07-06 15:53 UTC] hm2k (James Wade)
Description: ------------ http://svn.php.net/viewvc/pear/packages/Net_Whois/trunk/Whois.php?r1=300497&r2=300498 This change makes it more strict and less adaptable. Was there a problem with the previous regular expression? I tested it previously and it was a perfectly sufficient regular expression to match the string. I checked and was unable to replicate a "false positive". Do you have any examples?

Comments

 [2010-07-06 16:44 UTC] doconnor (Daniel O'Connor)
Bug #17540 GNICHOST is redundant perhaps?>
 [2010-07-06 20:08 UTC] hm2k (James Wade)
What? No, this issue is not covered by that bug. It's not even related.
 [2010-07-21 02:45 UTC] hm2k (James Wade)
I've discovered some kind of looping bug with domains on the "whois.tucows.com" whois server. At the moment I can't diagnose the bug properly as I very quickly hit my daily limits and am presented with the following error: IP Address: ??.???.???.??? Maximum Daily connection limit reached. Lookup refused. What seems to happen is it keeps looking up the domain until the script times out...
 [2010-07-23 02:13 UTC] kguest (Ken Guest)
-Assigned To: +Assigned To: kguest
 [2010-07-23 02:14 UTC] kguest (Ken Guest)
-Roadmap Versions: +Roadmap Versions: 1.0.4
I've reverted the change.
 [2010-07-23 02:26 UTC] kguest (Ken Guest)
-Status: Assigned +Status: Closed
This bug has been fixed in SVN. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.
 [2010-07-23 03:26 UTC] hm2k (James Wade)
-Status: Closed +Status: Open
I see no fix for this in the SVN.
 [2010-07-23 13:25 UTC] kguest (Ken Guest)
-Status: Assigned +Status: Closed
This bug has been fixed in SVN. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.