Skip to content

default couchdb response is 201 on successful POST#4

Merged
eyedol merged 1 commit intoushahidi:masterfrom
mandric:master
Dec 6, 2011
Merged

default couchdb response is 201 on successful POST#4
eyedol merged 1 commit intoushahidi:masterfrom
mandric:master

Conversation

@mandric
Copy link
Copy Markdown
Collaborator

@mandric mandric commented Nov 28, 2011

would make my life easier...

thanks!

eyedol added a commit that referenced this pull request Dec 6, 2011
default couchdb response is 201 on successful POST
@eyedol eyedol merged commit 53fc458 into ushahidi:master Dec 6, 2011
@eyedol
Copy link
Copy Markdown
Collaborator

eyedol commented Dec 6, 2011

Actually, this is the correct status code for a successful post request.

@mandric
Copy link
Copy Markdown
Collaborator Author

mandric commented Dec 7, 2011

I would argue either 200 or 201 is valid in this scenario. http://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html

But it is open to interpretation and if it conflicts with the SMSSync philosophy no big deal...

@mandric
Copy link
Copy Markdown
Collaborator Author

mandric commented Dec 7, 2011

Oh duh. Didn't realize you merged it! I'm a noise bot. Thanks!

@eyedol
Copy link
Copy Markdown
Collaborator

eyedol commented Dec 8, 2011

Yeah I merged your chances. I'll will have to test it tough. Actually, I'll make it either a 200 or 201 is a successful status code.

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