[CONTROLLER-1191] Restconf PUT to top-level node fails Created: 10/Mar/15 Updated: 25/Jul/23 Resolved: 16/Mar/15 |
|
| Status: | Resolved |
| Project: | controller |
| Component/s: | mdsal |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | ||
| Reporter: | Tom Pantelis | Assignee: | Tony Tkacik |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| External issue ID: | 2805 |
| Priority: | High |
| Comments |
| Comment by Tom Pantelis [ 10/Mar/15 ] |
|
If you do a PUT to a top-level node, an exception is thrown: aylight.controller.sal-rest-connector - 1.2.0.SNAPSHOT | | Error parsing json input This is easily reproducible with the toaster, eg: http://localhost:8181/restconf/config/toaster:toaster { } Also the CDS 3-node integration tests that use the car/people models are broken as well. |
| Comment by Tom Pantelis [ 10/Mar/15 ] |
|
Also doing POST to http://localhost:8181/restconf/config/, gets a different exception: java.lang.NullPointerException This is because the identifier portion of the URL was not specified. This is supposed to be allowed for POST (it used to work anyway). Perhaps this should be a separate bug... |
| Comment by Tom Pantelis [ 10/Mar/15 ] |
|
It looks like this may have been broken by https://git.opendaylight.org/gerrit/#/c/16209/2. |
| Comment by Tony Tkacik [ 10/Mar/15 ] |
|
Yeah, in patch you mentioned is bug, which provides JSON failure for top-level element, |
| Comment by Tom Pantelis [ 10/Mar/15 ] |
|
ok - so there's progress - that's good (In reply to Tony Tkacik from comment #4) |
| Comment by Tony Tkacik [ 10/Mar/15 ] |
|
https://git.opendaylight.org/gerrit/#/c/16136/ This fixes PUT for JSON inputs |