Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding POST style parameters for DELETE request. #448

Merged
merged 2 commits into from
Jul 30, 2013

Conversation

dquimper
Copy link
Contributor

Hi,

Grape used to accept POST style parameters for DELETE request. So I'm adding it back here.

However, I know that the HTTP spec doesn't explicitly disallow request bodies (where POST parameters go) for DELETE requests and many server implementations ignore it and you may run into issues with proxies dropping the content. So I'll leave it to you to decide if you which too add it.

References:
http://www.quora.com/Is-there-any-way-to-pass-parameters-for-HTTP-DELETE-method
http://stackoverflow.com/questions/299628/is-an-entity-body-allowed-for-an-http-delete-request

@dblock
Copy link
Member

dblock commented Jul 30, 2013

I'll take it. Can you please also update CHANGELOG?

@dquimper
Copy link
Contributor Author

There you go.

dblock added a commit that referenced this pull request Jul 30, 2013
Adding POST style parameters for DELETE request.
@dblock dblock merged commit b6a5778 into ruby-grape:master Jul 30, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants