Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Got these while running my thousands of test URLs.
Warning: fwrite(): 758 is not a valid stream resource in httprl_send_request() (line 569 of drupal7\sites\all\modules\httprl\httprl.module).
Comment | File | Size | Author |
---|---|---|---|
#10 | httprl-1427966-10-fwrite-issue.patch | 618 bytes | mikeytown2 |
#8 | httprl-14227966-6-fix-fwrite-issue.patch | 614 bytes | klokie |
#5 | httprl-14227966-5-fix-fwrite-issue.patch | 710 bytes | mikeytown2 |
#4 | httprl-14227966-4-fix-fwrite-issue.patch | 2 KB | mikeytown2 |
Comments
Comment #0.0
hass CreditAttribution: hass commentedff
Comment #0.1
hass CreditAttribution: hass commentedOne issue moved to extra case having a repro case.
Comment #1
hass CreditAttribution: hass commentedRepro case is currently unknown
Comment #2
hass CreditAttribution: hass commentedRepro cases found:
Comment #3
hass CreditAttribution: hass commentedThese URLs have missing status code in the result, too!
Comment #4
mikeytown2 CreditAttribution: mikeytown2 commentedThis is a pretty odd case, good job finding it. This patch should fix it; it has been committed.
Comment #5
mikeytown2 CreditAttribution: mikeytown2 commentedempty() should be === FALSE http://php.net/array-search
This patch has been committed.
Comment #6
hass CreditAttribution: hass commentedResults looks good. All status codes "-1 request timed out" are inside. THX
Comment #8
klokie CreditAttribution: 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 CreditAttribution: klokie commentedComment #10
mikeytown2 CreditAttribution: 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 CreditAttribution: mikeytown2 commentedMarking this postponed until I can get feedback.
Comment #12
mikeytown2 CreditAttribution: mikeytown2 commentedClosing issue as I can not reproduce it with the latest dev.
Comment #12.0
mikeytown2 CreditAttribution: mikeytown2 commented#1429272