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

Cases of PUT / DELETE HTTP methods #40

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

gigaga
Copy link

@gigaga gigaga commented Mar 10, 2015

Hi,

in Cases of PUT / DELETE HTTP methods, in REST architecture, there is not response from backend in payload (only HTTP status), however, actually, the response is checked if it is an object. If not, an error is thrown "The Properties argument must be an object.".
It seems that is not correct.

Thx.

@passy
Copy link
Contributor

passy commented Apr 20, 2015

Sorry for the late response, @gigaga. Could you provide a test case for which this behavior is desired? I don't see from your description how this would work, but I'm sure a unit test would clarify that.

@gigaga
Copy link
Author

gigaga commented Apr 20, 2015

I have not test case :-(.
However, the problem is really easy to reproduce and unterstand. Indeed, when the hal backend server does not provide any body in the response (in case of Delete and Put verbs), there is an error. It should be possible to have not body in the response. Normally, for Put and Delete Http methods, only the Http code status is necessary.

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

Successfully merging this pull request may close these issues.

2 participants