> -----Original Message----- > From: rohityada...@gmail.com [mailto:rohityada...@gmail.com] On Behalf > Of Rohit Yadav > Sent: 30 January 2013 10:30 > To: cloudstack-dev@incubator.apache.org > Subject: Re: [DISCUSS] DeployVirtualMachine userdata enhancements > > 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 > Per HTTP specs for update api we could use PUT. For delete apis we could stick with DELETE. That is true REST.
> > 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/DeployVirtual > Machi > >>>ne+userdata+enhancements > >>> along with a JIRA ticket > >>>https://issues.apache.org/jira/browse/CLOUDSTACK-1086 > >>> > >>> > >>> Please review and comment > >>> > >>> Hari Kannan > >
