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

in POST v2/op/update (batch op) if we do not use entity type (optional) it is assumed as empty #2685

Closed
iariasleon opened this issue Nov 16, 2016 · 3 comments
Assignees
Milestone

Comments

@iariasleon
Copy link
Contributor

in POST v2/op/update (batch op) if we do not use entity type (optional) it is assumed as empty. It is related to #2665.

update batch op request

url: POST http://localhost:1026/v2/op/update
headers:
    Content-Type: application/json
    Fiware-Service: test_op_update_entities_type_error
    Fiware-ServicePath: /test
payload: {"entities": [{"temperature": {"type": "celsius", "value": "4", "metadata": {"alarm": {"type": "hot", "value": "warning"}}}, "id": "room_4"}], "actionType": "APPEND"}

update response

http code: 400 - Bad Request
headers:
   Connection: Keep-Alive
   Content-Length: 85
   Content-Type: application/json
   Fiware-Correlator: 443fdb4a-ac1c-11e6-9947-005056a20feb
   Date: Wed, 16 Nov 2016 16:46:48 GMT
payload: {"error":"BadRequest","description":"entity type length: 0, min length supported: 1"}

expected response

http code: 204 - No Content
@iariasleon iariasleon added this to the 1.6.0 milestone Nov 16, 2016
@crbrox crbrox self-assigned this Nov 23, 2016
@fgalan
Copy link
Member

fgalan commented Nov 23, 2016

Fixed by PR #2707

@fgalan fgalan assigned iariasleon and unassigned crbrox Nov 25, 2016
@fgalan
Copy link
Member

fgalan commented Nov 25, 2016

@iariasleon ready for re-check, pls.

@iariasleon iariasleon reopened this Nov 28, 2016
@iariasleon
Copy link
Contributor Author

te-tested and LGTM

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

No branches or pull requests

3 participants