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

add HTTP support for PUT and DELETE #312

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
7 participants
@firefly2442
Contributor

firefly2442 commented Nov 2, 2012

I did some rudimentary testing and it seems to work. I have little experience with REST API's however, so someone else may want to test this who has additional experience.

Fixes issue #257

@ghost ghost assigned LaurentGomila Sep 14, 2013

@@ -109,6 +109,8 @@ std::string Http::Request::prepare() const
case Get : method = "GET"; break;
case Post : method = "POST"; break;
case Head : method = "HEAD"; break;
case Put : method = "PUT"; break;

This comment has been minimized.

@kimci86

kimci86 May 1, 2014

Contributor

There is a tab character after the colon instead of spaces.

@kimci86

kimci86 May 1, 2014

Contributor

There is a tab character after the colon instead of spaces.

@binary1248

This comment has been minimized.

Show comment
Hide comment
@binary1248

binary1248 May 22, 2014

Member

Modified version in #607.

Member

binary1248 commented May 22, 2014

Modified version in #607.

@binary1248 binary1248 closed this May 22, 2014

@mantognini mantognini modified the milestones: 2.x, 2.2 May 22, 2014

@eXpl0it3r eXpl0it3r added the resolved label May 22, 2014

@eXpl0it3r eXpl0it3r modified the milestone: 2.2 May 22, 2014

@eXpl0it3r eXpl0it3r added s:rejected and removed s:accepted labels Jul 7, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment