Request body parser should reject body with invalid elements
Bug #814518 reported by
Salvatore Orlando
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
neutron |
Fix Released
|
Low
|
Salvatore Orlando | ||
quantum (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Bug Description
Currently the request body parser ignores invalid data in request body if a request body is not required.
The API should instead reject the request and return a 400 error.
Related branches
lp:~salvatore-orlando/neutron/bug814518
- dan wendlandt: Approve
- Brad Hall (community): Approve
-
Diff: 72 lines (+31/-3)3 files modifiedquantum/api/api_common.py (+7/-1)
tests/unit/test_api.py (+21/-0)
tests/unit/testlib_api.py (+3/-2)
Changed in quantum: | |
assignee: | nobody → Salvatore Orlando (salvatore-orlando) |
Changed in quantum: | |
importance: | Undecided → Low |
Changed in quantum: | |
milestone: | none → diablo-3 |
Changed in quantum: | |
milestone: | diablo-3 → diablo-4 |
Changed in quantum: | |
status: | New → In Progress |
Changed in quantum: | |
status: | In Progress → Fix Committed |
Changed in quantum: | |
status: | Fix Committed → Fix Released |
Changed in quantum (Ubuntu): | |
status: | New → Fix Released |
To post a comment you must log in.
According to Salvatore, this is the last bug that needs to be fixed to get the existing unit tests running clean.
Salvatore, an you specify the unit test(s) do not pass because of this test? If so I can probably do a quick fix.
Also, what's your definition of "invalid data" here. Something that does not parse, or something that contains an unexpected parameter?