Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

Add body to curl DELETE request if one is specified. #6318

Closed
wants to merge 1 commit into from
Closed

Add body to curl DELETE request if one is specified. #6318

wants to merge 1 commit into from

Conversation

lucian303
Copy link
Contributor

The header will be there. Fixes issue #6315 (#6315).

When providing a request body for a DELETE request, Zend\Http\Client will add the 'Content-Length' header but will not add and send the actual body resulting in a curl call that hangs for a very long period of time. The client should send both a body and the header since DELETE requests can have a body. From the IETF spec: "The presence of a message-body in a request is signaled by the inclusion of a Content-Length or Transfer-Encoding header field in the request's message-headers." -- http://tools.ietf.org/html/rfc2616

@Ocramius Ocramius self-assigned this May 23, 2014
@Ocramius Ocramius added this to the 2.3.2 milestone May 23, 2014
Ocramius added a commit that referenced this pull request Jul 27, 2014
@Ocramius
Copy link
Member

@lucian303 merged, thanks

@Ocramius Ocramius closed this in 8472ba1 Jul 27, 2014
gianarb pushed a commit to zendframework/zend-http that referenced this pull request May 15, 2015
gianarb pushed a commit to zendframework/zend-http that referenced this pull request May 15, 2015
…rk/zf2#6315-add-body-to-curl-delete-request' into develop

Forward port zendframework/zendframework#6318
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants