'delete' API call wrongly documented in README about needle

[tomas]

Oops! Fix it right away, sorry mate.

[jeroenr]

Awesome work! No problem. I got bit by it and wanted to prevent others
getting bit by it as well. Thanks a lot for this library. It's great and
easy to use. I really needed a wrapper around Node's low level HTTP lib.

On Tue, Jul 9, 2013 at 4:22 PM, Tomás Pollak [email protected]:

Oops! Fix it right away, sorry mate.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/30#issuecomment-20677133
.

[jeroenr]

On second thought, I believe you should prevent sending data in a delete. According to the HTTP spec it's not allowed to send data.

[tomas]

I'm pretty sure the HTTP spec doesn't state whether you need to or are forbidden from sending data on a delete request. Where did you read that?

[jeroenr]

You are right. I've been misreading. It's just that it''s not common to do
it and some web servers ignore body in DELETE request, but you are right.
It's definitely allowed. Apologies for the confusion.

On Wed, Jul 10, 2013 at 3:54 PM, Tomás Pollak [email protected]:

I'm pretty sure the HTTP spec doesn't state whether you need to or are
forbidden from sending data on a delete request. Where did you read that?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/30#issuecomment-20743401
.

[tomas]

No problemo mate.