Closed (fixed)
Project:
HTTP Parallel Request & Threading Library
Version:
7.x-1.8
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
25 Dec 2012 at 18:50 UTC
Updated:
10 Jan 2013 at 10:50 UTC
Jump to comment: Most recent file
Comments
Comment #1
hass commentedAfter more thinking this was on my dev box before we ignored -1004, but we need to handle this. Otherwise last_checked is never set and we are checking this links with every cron run again and again. I missed this. It's like end endless loop, but endless check. Is -1004 a malfunction?
If this "host does not exists" situation cannot catched inside httprl - I may need to create a new linkchecker release. :-(
Comment #2
mikeytown2 commentedhttp://www.klug-suchen.de gives me a -2 (max redirects exhausted)
With D6 & D7 drupal_http_request() I get:
So when you're using Drupal core you will still need to parse the error text.
The attached patch has been committed to 6.x & 7.x. Thanks for the bug report. Keep them coming :)
Note: Where -1004 came from #193073-11: drupal_http_request - socket not initialized
Comment #3
hass commentedBad news... The text is german on my system. :-(
Only
network_getaddresses: getaddrinfo failed:is the same.Comment #4
hass commentedOne more example of inconsistencies http://stackoverflow.com/questions/8210099/php-php-network-getaddresses-...
Comment #5
mikeytown2 commentedThanks :)
This patch has been committed.
Comment #6
hass commentedhttp://php.net/manual/en/function.fsockopen.php
Comment #7
hass commentedCross post...
Comment #8
mikeytown2 commentedstream_socket_client is the function that is throwing the error. It's used in D7 as well. D6 does use fsockopen though.
Comment #9
mikeytown2 commentedMoving this back to fixed.
Comment #10
hass commentedWell, but it may return the same warning, maybe just not documented? not tested... I'm not sure how reliable the new logic works...
How about calling dns_get_record() just to make it more reliable?
Comment #11
mikeytown2 commentedThat function is only available on windows if PHP >= 5.3
Also noted is this issue #1325662: Usage and/or caching of gethostbyname
I'm not about to get into the business of implementing my own DNS client, even though that would give HTTPRL some big speed improvements when it comes to checking a lot of URLs.
I will be using "php_network_getaddresses: getaddrinfo failed:" for now. Like usual, I'm open to other patches, just not going to implement this my self.
Comment #12
hass commentedOk, should be fine for now.
Comment #13
hass commentedPatch works well for me.
Comment #14.0
(not verified) commentedg