Closed (cannot reproduce)
Project:
HTTP Parallel Request & Threading Library
Version:
6.x-1.7
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
4 Feb 2012 at 03:18 UTC
Updated:
21 Dec 2012 at 22:21 UTC
Jump to comment: Most recent file
Comments
Comment #0.0
hass commentedff
Comment #0.1
hass commentedOne issue moved to extra case having a repro case.
Comment #1
hass commentedRepro case is currently unknown
Comment #2
hass commentedRepro cases found:
Comment #3
hass commentedThese URLs have missing status code in the result, too!
Comment #4
mikeytown2 commentedThis is a pretty odd case, good job finding it. This patch should fix it; it has been committed.
Comment #5
mikeytown2 commentedempty() should be === FALSE http://php.net/array-search
This patch has been committed.
Comment #6
hass commentedResults looks good. All status codes "-1 request timed out" are inside. THX
Comment #8
klokie commentedI'm having the same issue with 6.x-1.7. I've backported the patch in #5. Hope it's ok. Thanks!
Comment #9
klokie commentedComment #10
mikeytown2 commentedIn the latest dev version the $streams variable went away due to this issue getting fixed #1697244: Connection refused by destination. Write. curl works . As a result, the patch that you added isn't going to do the right thing. Also noted is the 6.x and 7.x versions of httprl should always be exactly the same.
http://drupalcode.org/project/httprl.git/blob/refs/heads/6.x-1.x:/httprl...
http://drupalcode.org/project/httprl.git/blob/refs/heads/7.x-1.x:/httprl...
Trying the example in #2 against the latest dev does not repo this.
Can you tell me how you are able to reproduce this bug? Here is a patch for 6.x & 7.x dev that adds this check back in. I need to know that this bug exists in the latest dev, and the attached patch fixes it before I commit this.
Comment #11
mikeytown2 commentedMarking this postponed until I can get feedback.
Comment #12
mikeytown2 commentedClosing issue as I can not reproduce it with the latest dev.
Comment #12.0
mikeytown2 commented#1429272