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  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 29 - 7 = ?

 
 [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.