Re: [Architecture] RESTFul API for App Manager - Design discussion

2016-05-01 Thread Sanjeewa Malalgoda
Hi Frank, Thilini, Really sorry that i missed this thread and taking long time to reply. As i understood this is more like a application implementation level thing and its not related to authentication and security. To be clear on this, security, authentication or authorization may depend on user

Re: [Architecture] RESTFul API for App Manager - Design discussion

2016-04-27 Thread Frank Leymann
Sanjeewa (or Prabath) may have guidance on this... Best regards, Frank 2016-04-27 11:11 GMT+02:00 Thilini Shanika : > + Frank > > On Wed, Apr 27, 2016 at 2:36 PM, Thilini Shanika > wrote: > >> Hi, >> >> We have a little concern regarding the permission

Re: [Architecture] RESTFul API for App Manager - Design discussion

2016-04-27 Thread Thilini Shanika
+ Frank On Wed, Apr 27, 2016 at 2:36 PM, Thilini Shanika wrote: > Hi, > > We have a little concern regarding the permission check in OAuth secured > APIs in APPM. > > Currently, all the available APIs are attached with relevant OAuth 2 > scopes, but there are some cases where

Re: [Architecture] RESTFul API for App Manager - Design discussion

2016-04-27 Thread Thilini Shanika
Hi, We have a little concern regarding the permission check in OAuth secured APIs in APPM. Currently, all the available APIs are attached with relevant OAuth 2 scopes, but there are some cases where we cannot manage permissions using scopes only. For example, in application lifecycle change

Re: [Architecture] RESTFul API for App Manager - Design discussion

2016-04-26 Thread Manuranga Perera
agreed Gayan. If we have following, it will be clearer. I changed 'apps' to ' installations' DELETE http://localhost:9763/api/appm/storeadmin/v1.0/users/ admin/installations/cec2027d-2dd6-4826-97c5-33be4eb83ae1 On Tue, Apr 26, 2016 at 12:33 AM, Gayan Gunarathne wrote: > > > On

Re: [Architecture] RESTFul API for App Manager - Design discussion

2016-04-25 Thread Gayan Gunarathne
On Mon, Apr 25, 2016 at 6:48 PM, Manuranga Perera wrote: > DELETE is verb to un-link resources. > Eg: DELETE /user/starred/manu/product-appm [1] will un-link me form appM. > But it doesn't mean AppM repo will be deleted. > See the url here. That make sense it deletes the starred

Re: [Architecture] RESTFul API for App Manager - Design discussion

2016-04-25 Thread Manuranga Perera
DELETE is verb to un-link resources. Eg: DELETE /user/starred/manu/product-appm [1] will un-link me form appM. But it doesn't mean AppM repo will be deleted. [1] https://developer.github.com/v3/activity/starring/#unstar-a-repository On Mon, Apr 25, 2016 at 6:55 AM, SajithAR Ariyarathna

Re: [Architecture] RESTFul API for App Manager - Design discussion

2016-04-25 Thread SajithAR Ariyarathna
[+ Frank] On Mon, Apr 25, 2016 at 11:49 AM, SajithAR Ariyarathna wrote: > ... "uninstall" is NOT a REST, even though it looks like REST. > > "uninstall" does not looks like REST. It looks like RPC. > > On Mon, Apr 25, 2016 at 10:43 AM, Ruwan Abeykoon wrote:

Re: [Architecture] RESTFul API for App Manager - Design discussion

2016-04-25 Thread SajithAR Ariyarathna
> > ... "uninstall" is NOT a REST, even though it looks like REST. "uninstall" does not looks like REST. It looks like RPC. On Mon, Apr 25, 2016 at 10:43 AM, Ruwan Abeykoon wrote: > I disagree Manu, The point is this is not "DELETE" action on a "resource". > This is an action

Re: [Architecture] RESTFul API for App Manager - Design discussion

2016-04-24 Thread Ruwan Abeykoon
I disagree Manu, The point is this is not "DELETE" action on a "resource". This is an action to remove a link between two resources. But one oth that "Resource" is just a virtual resource(which is not under control of the system). So I think having DELETE HTTP verb is wrong in this. Cheer, Ruwan

Re: [Architecture] RESTFul API for App Manager - Design discussion

2016-04-24 Thread Manuranga Perera
I think, even if it's not immediately affected (asynchronous), you still have to use the correct HTTP verb. ___ Architecture mailing list Architecture@wso2.org https://mail.wso2.org/cgi-bin/mailman/listinfo/architecture

Re: [Architecture] RESTFul API for App Manager - Design discussion

2016-04-24 Thread Ruwan Abeykoon
>>Can we we something like: >>DELETE http://localhost:9763/api/appm/storeadmin/v1.0/users/admin/apps/cec2027d-2dd6-4826-97c5-33be4eb83ae1 >> -1, This has been discussed that the "uninstall" is NOT a REST, even though it looks like REST. The "DELETE" should be effected on a resource. However

Re: [Architecture] RESTFul API for App Manager - Design discussion

2016-04-20 Thread Udara Liyanage
On Wed, Apr 20, 2016 at 12:36 PM, Manuranga Perera wrote: > Instead: > > POST http://localhost:9763/api/appm/storeadmin/v1.0/apps/uninstallation > { >"type": "user", >"typeIds": ["admin"], > "appId": "cec2027d-2dd6-4826-97c5-33be4eb83ae1" > } > > Can we we something

Re: [Architecture] RESTFul API for App Manager - Design discussion

2016-04-20 Thread Manuranga Perera
Instead: POST http://localhost:9763/api/appm/storeadmin/v1.0/apps/uninstallation { "type": "user", "typeIds": ["admin"], "appId": "cec2027d-2dd6-4826-97c5-33be4eb83ae1" } Can we we something like: DELETE

Re: [Architecture] RESTFul API for App Manager - Design discussion

2016-04-20 Thread Lahiru Cooray
Hi all, Here with I'm sharing the finalized swagger definitions for your reference. On Thu, Mar 31, 2016 at 9:15 AM, Lahiru Cooray wrote: > Hi Sanjiva, > Herewith I have attached the pdf's and yes, we referred the WSO2 REST API > design guideline doc (attached) and also the

Re: [Architecture] RESTFul API for App Manager - Design discussion

2016-03-30 Thread Sanjiva Weerawarana
Lahiru when sharing Google docs to this public list you need to PDF and attach as well .. non-WSO2 people can't read this. I assume this is compatible with the REST API design guidelines doc we wrote right? On Wed, Mar 30, 2016 at 10:06 AM, Lahiru Cooray wrote: > Hi All, > >

[Architecture] RESTFul API for App Manager - Design discussion

2016-03-29 Thread Lahiru Cooray
Hi All, We are planing to implement a RESTFul API to expose the App Manager functionality. Initially we are focussing on replacing AppM Mobile API's (Store/Publisher) . Please refer the proposed new set of API's design [1] (and comparison with existing API's [2]) and comment your suggestions [1]