I tried to integrate the module into linkchecker... Added 1000 urls from d.o into httprl_request()... We need to solve #1268096: Implement a rate limiter asap.

Incomplete responds array:
2012-01-28_145332.png

I guess my action killed my local server I fired the stuff, too.

So what's the problem?

  1. Status == "Done"
  2. Code is missing. Really need this value!
  3. Status_message is missing.
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hass’s picture

Issue summary: View changes

Embedded link is defect

hass’s picture

Issue summary: View changes

foo

hass’s picture

Priority: Normal » Critical

I've seen one more variant where the "error" and "code" was below "request". This means $foo->request->error and $foo->request->code and missing in root of Class. It must always be $foo->code and $foo->error. $foo->code is required in all cases, error is optional.

hass’s picture

Issue summary: View changes

fff

hass’s picture

hass’s picture

FileSize
20.54 KB

Example dump for #1 bug

2012-01-29_032521.png

hass’s picture

Added 2 lines (Line 350ff) in function httprl_send_request(): $result->code and $result->error

    // Connection was never made.
    else {
      $result->status = 'Connection not made.';
      $result->code = (int) $request->code;
      $result->error = $request->error;
      $counter++;
      return FALSE;
    }
hass’s picture

Title: Responds array incomplete / inconsistent » Responds object incomplete / inconsistent
Status: Active » Needs review
FileSize
592 bytes

Patch for #4. It does not fix the redirect inconsistency bug(s).

hass’s picture

Re-attach patch with changed name as %2F cause problems on d.o. The (int) is there as it seems to be possible that this is sometimes not an int what is not allowed, but I've seen it... no idea why.

hass’s picture

With more thinking it may be better to refactor/review the full logic to prevent such consistencies in general. What i've done here was a quick fix that may break in other situations... Not sure.

mikeytown2’s picture

Status: Needs review » Fixed
FileSize
1.31 KB

Pushed the casting to an INT higher up in the code. Patch below committed to 6.x & 7.x.

hass’s picture

Does this also fix the "redirect" inconsistencies compared to core?

mikeytown2’s picture

I think it might, only one way to find out ;)
#1320222-3: Bring in other drupal patches

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

Issue summary: View changes

foo