On Tue, Jan 29, 2013 at 8:30 PM, Nitin Mehta <nitin.me...@citrix.com> wrote: > Good point Rohit, but I suggest making only those API's POST which create > a resource (like deployVm here) and not all of them.
> Ideally, we need to be using the right http methods for a true restful web > service. Nice suggestion, while we can use GET for all read/list apis and POST for rest kind of apis would work; so we should use correct HTTP request type, for example for general crud apps: create apis: POST list/read apis: GET update apis: PUT or POST delete apis: DELETE or POST > Also about the length part, how then we allow user data to be of size 2kb > currently ? It's was a known issue, while it would work with most modern age browsers this was paid less attention. As Mice suggests, should we increase length of tags? Regards. > > On 30/01/13 9:43 AM, "Rohit Yadav" <bhais...@apache.org> wrote: > >>+1 But if we are sending base64 encoded userdata as part of POST, we >>can increase the limit even further. >> >>-1 If userdata will be sent as part of GET query, based on the rfc [1] >>and from the widely used ugliest web browser [2] the GET url length >>should be <= 2000 (we're already exceeding that how we send userdata >>request at present). >> >>Suggestion: Make POST as the default way of requesting apis from mgmt >>server. >> >>[1] http://www.faqs.org/rfcs/rfc2616.html >>[2] http://support.microsoft.com/kb/q208427 >> >>Regards. >> >>On Tue, Jan 29, 2013 at 7:40 PM, Hari Kannan <hari.kan...@citrix.com> >>wrote: >>> Hello All, >>> >>> I wish to propose increasing the size of userdata to be passed along >>>with DeployVirtualMachine API - I have added some info here >>>https://cwiki.apache.org/confluence/display/CLOUDSTACK/DeployVirtualMachi >>>ne+userdata+enhancements >>> along with a JIRA ticket >>>https://issues.apache.org/jira/browse/CLOUDSTACK-1086 >>> >>> >>> Please review and comment >>> >>> Hari Kannan >
