[CONTROLLER-1341] Adding a trailing slash to certain RestConf URLs results in HTTP 400 Bad Request Created: 27/May/15 Updated: 21/Aug/15 Resolved: 21/Aug/15 |
|
| Status: | Resolved |
| Project: | controller |
| Component/s: | restconf |
| Affects Version/s: | Helium |
| Fix Version/s: | None |
| Type: | Bug | ||
| Reporter: | Ryan Goulding | Assignee: | Jan Hajnar |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| External issue ID: | 3395 |
| Description |
|
The working scenario is first detailed below: URL -> http://localhost:8181/restconf/operations/toaster:make-toast This results in HTTP 204, success. However, if you add a trailing slash to the URL it does not: This results in HTTP 400, Bad Request. The trailing slash should not result in HTTP 400. This is fixed in stable/lithium, but is a bug in stable/helium. |
| Comments |
| Comment by Jan Hajnar [ 26/Jun/15 ] |
| Comment by Colin Dixon [ 19/Aug/15 ] |
|
This missed Helium. Is it also present in Lithium? Beryllium? |
| Comment by Jan Hajnar [ 20/Aug/15 ] |
|
Hi, This patch was for helium only. This issue is fixed in lithium/beryllium. Jan. |
| Comment by Colin Dixon [ 21/Aug/15 ] |
|
This issue is already resolved in Lithium and beyond and as there are no more planned releases of Helium, there is no plan to fix it in Helium. |
| Comment by Colin Dixon [ 21/Aug/15 ] |
|
I closed it as wontfix, but it seems like something that might not be a bad thing to merge to stable/helium since it takes no real effort even thought it might not get shipped. |