Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.We shoudl alter acquia_agent_http_request() along the lines fo the alteration to drupal_http_request() here:
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | 491600_8_Make_Content_Length_Optional_And_Remove_Self_Check.patch | 2.25 KB | Anonymous (not verified) |
| #7 | 491600_7_Make_Content_Length_Optional_And_Remove_Self_Check.patch | 2.26 KB | Anonymous (not verified) |
| #1 | nocheck-491600-1.patch | 1.57 KB | pwolanin |
Comments
Comment #1
pwolanin commentedmaybe just this?
We don't really need to add the check-again code as core does.
Comment #2
pwolanin commentedComment #3
David_Rothstein commentedThe code makes sense, and I tried it out and acquia_agent_http_request() still seems to work.
It looks like there are some other minor improvements in drupal_http_request() that we don't have, but they look unimportant enough to skip. Overall, it's really a shame that we have to keep a hacked version of this function...
Comment #4
pwolanin commentedyes, but no other way to check the ssl cert
Comment #5
pwolanin commentedHere's one core issue, but essentially blocked on Crell's mythical handlers in core: http://drupal.org/node/64866
Comment #6
pwolanin commentedmaybe we shoudl pull this in too? http://drupal.org/node/280240
Comment #7
Anonymous (not verified) commentedIncludes #1 and #6 with modification listed from http://drupal.org/node/280240#comment-1709162
Comment #8
Anonymous (not verified) commentedComment #9
pwolanin commentedthanks Joshua - committed and then branched to DRUPAL-6--1