These sites seems to be up and running, but are broken this way in reports:

http://www.jobrobot.de/	 -10060	 Connection timed out. If you believe this is a false error, turn off async_connect in the httprl options array and try again. Read.
http://www.europas-mitte-um-1000.de.vu/	 -10060	 Connection timed out. If you believe this is a false error, turn off async_connect in the httprl options array and try again. Write.

Some others reports are correct, compared to wget --spider [url] (Connection timed out):

http://www.science-tech.nmstc.ca/english/index.cfm
http://www.aviation.nmstc.ca/Eng/english_home.html
http://www2.imj.org.il/eng/branches/rockefeller/final/index.html
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hass’s picture

PHP 5.3.1

mikeytown2’s picture

Status: Active » Fixed
FileSize
4.91 KB

Repo Code:

$urls = array(
  'http://www.science-tech.nmstc.ca/english/index.cfm',
  'http://www.aviation.nmstc.ca/Eng/english_home.html',
  'http://www2.imj.org.il/eng/branches/rockefeller/final/index.html',
  'http://www.jobrobot.de/',
  'http://www.europas-mitte-um-1000.de.vu/',
);

httprl_request($urls, array('method' => 'HEAD', 'max_redirects' => 0));
$request = httprl_send_request();
echo httprl_pr($request);

Note that if I move

  'http://www.jobrobot.de/',
  'http://www.europas-mitte-um-1000.de.vu/',

to the top of the array I get 200's for these URLs.

That tells me that the issue is with short-cut timeouts most likely.



After digging deeper I found an issue that doesn't effect link checker but it does effect the example code. The Host header is not set correctly when passing in an array of URLs to httprl_request().

I've attached a patch that fixes both of these issues. Marking this issue as fixed. Let me know if you find any more bugs :)

hass’s picture

Great, hopefully there are no more issues :-)

hass’s picture

/**
 * Error code indicating that this request made by stream_select() couldn't
 * open a read and/or write steam after a minimum of ~2.5 seconds.
 */

Isn't this not 10s now? Aside, there is a typo in "steam".

mikeytown2’s picture

This patch has been committed. It fixes the typos :)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.