tapis-project / tapis-apps

BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Apps: Potential tapipy issue with appId vs Id #4

Closed scblack321 closed 2 years ago

scblack321 commented 2 years ago

POST endpoint json request body uses field name id. PUT endpoint uses appId in the path and allows for field name id in the request body. For the PUT the id in the request body is ignored. This appears to cause a problem with tapipy. Possibly because tapipy is creating a single dictionary from path parameters and fields in the request body. Nathan Freeman and Tyler Clemens discovered this while implementing a CLI using tapipy. They found they needed to use id in the json for the tapipy createApp call and appId for the json in the tapipy putApp call. Possibly the issue lies in the CLI implementation. NOTE: This probably applies to Systems as well.

NOTE: Fixed by Tyler in CLI code