Re: [Carbon-dev] WSRealmBuilder API changes..
Hi, Fixed the realm properly. Now this is usable in non-carbon environment. tx, dimuthul On Fri, Mar 4, 2011 at 9:39 AM, Prabath Siriwardana wrote: > Hi Dimuthu, > > Thanks for getting more deep in to this.. > > Actually this methods was in 3.0.1 > > > WSRealmBuilder.createWSRealm(serverUrl, cookie, configContext); > > Thanks & regards, > > -Prabath > > > On Fri, Mar 4, 2011 at 9:32 AM, Dimuthu Leelarathne > wrote: > > > > Hi, > > > > The old method available was, > > WSRealmBuilder.createWSRealm(realmProperties, configurationContext, > tenantID) > > > > TenantId is meaningless here. Why? Because shouldn't expose an API where > IHC tenant can see some other tenant's data. So we cannot go back to old > API. > > > > We should have two methods as follows, > > WSRealmBuilder.createWSRealm(realmProperties, configurationContext) > > WSRealmBuilder.createWSRealm(realmProperties) > > > > If super tenant wants to access other tenant's data we can discuss it > (consider pros and cons) and make it available for super tenant only. > > > > Thanks, > > Dimuthu > > > > > > > > > > > > > > On Fri, Mar 4, 2011 at 9:03 AM, Prabath Siriwardana > wrote: > >> > >> Can we get the same old API back.. if it doesn't violate the reason it's > been removed previously..? > >> Thanks & regards, > >> -Prabath > >> > >> On Fri, Mar 4, 2011 at 8:54 AM, Dimuthu Leelarathne > wrote: > >>> > >>> Hi, > >>> > >>> Yes. You have a point. > >>> > >>> +1 for fixing it to get the configuration context. > >>> > >>> tx, > >>> dimuthu > >>> > >>> > >>> > >>> On Fri, Mar 4, 2011 at 8:50 AM, Prabath Siriwardana > wrote: > > I have a client out side the Carbon - which uses following API > > UserRealm realm = WSRealmBuilder.createWSRealm(serverUrl, cookie, > configContext); > But this code doesn't work with the current trunk - and the API been > changed.. > Can you please suggest how should I replace the above..? > > -- > Thanks & Regards, > Prabath > > http://blog.facilelogin.com > http://RampartFAQ.com > > ___ > Carbon-dev mailing list > Carbon-dev@wso2.org > http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev > > >>> > >> > >> > >> > >> -- > >> Thanks & Regards, > >> Prabath > >> > >> http://blog.facilelogin.com > >> http://RampartFAQ.com > > > > > > -- > Thanks & Regards, > Prabath > > http://blog.facilelogin.com > http://RampartFAQ.com > ___ Carbon-dev mailing list Carbon-dev@wso2.org http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev
Re: [Carbon-dev] WSRealmBuilder API changes..
Hi Dimuthu, Thanks for getting more deep in to this.. Actually this methods was in 3.0.1 WSRealmBuilder.createWSRealm(serverUrl, cookie, configContext); Thanks & regards, -Prabath On Fri, Mar 4, 2011 at 9:32 AM, Dimuthu Leelarathne wrote: > > Hi, > > The old method available was, > WSRealmBuilder.createWSRealm(realmProperties, configurationContext, tenantID) > > TenantId is meaningless here. Why? Because shouldn't expose an API where IHC tenant can see some other tenant's data. So we cannot go back to old API. > > We should have two methods as follows, > WSRealmBuilder.createWSRealm(realmProperties, configurationContext) > WSRealmBuilder.createWSRealm(realmProperties) > > If super tenant wants to access other tenant's data we can discuss it (consider pros and cons) and make it available for super tenant only. > > Thanks, > Dimuthu > > > > > > > On Fri, Mar 4, 2011 at 9:03 AM, Prabath Siriwardana wrote: >> >> Can we get the same old API back.. if it doesn't violate the reason it's been removed previously..? >> Thanks & regards, >> -Prabath >> >> On Fri, Mar 4, 2011 at 8:54 AM, Dimuthu Leelarathne wrote: >>> >>> Hi, >>> >>> Yes. You have a point. >>> >>> +1 for fixing it to get the configuration context. >>> >>> tx, >>> dimuthu >>> >>> >>> >>> On Fri, Mar 4, 2011 at 8:50 AM, Prabath Siriwardana wrote: I have a client out side the Carbon - which uses following API UserRealm realm = WSRealmBuilder.createWSRealm(serverUrl, cookie, configContext); But this code doesn't work with the current trunk - and the API been changed.. Can you please suggest how should I replace the above..? -- Thanks & Regards, Prabath http://blog.facilelogin.com http://RampartFAQ.com ___ Carbon-dev mailing list Carbon-dev@wso2.org http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev >>> >> >> >> >> -- >> Thanks & Regards, >> Prabath >> >> http://blog.facilelogin.com >> http://RampartFAQ.com > -- Thanks & Regards, Prabath http://blog.facilelogin.com http://RampartFAQ.com ___ Carbon-dev mailing list Carbon-dev@wso2.org http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev
Re: [Carbon-dev] WSRealmBuilder API changes..
Hi, On Fri, Mar 4, 2011 at 9:32 AM, Dimuthu Leelarathne wrote: > Hi, > > The old method available was, > WSRealmBuilder.createWSRealm(realmProperties, configurationContext, > tenantID) > > TenantId is meaningless here. Why? Because shouldn't expose an API where > IHC tenant can see some other tenant's data. So we cannot go back to old > API. > > We should have two methods as follows, > WSRealmBuilder.createWSRealm(realmProperties, configurationContext) > WSRealmBuilder.createWSRealm(realmProperties) > > Actually we have the above two methods in the API but the 1st method is useless because it is not implemented properly. Let me fix it properly. tx, dimuthu > If super tenant wants to access other tenant's data we can discuss it > (consider pros and cons) and make it available for super tenant only. > > Thanks, > Dimuthu > > > > > > > > On Fri, Mar 4, 2011 at 9:03 AM, Prabath Siriwardana wrote: > >> Can we get the same old API back.. if it doesn't violate the reason it's >> been removed previously..? >> >> Thanks & regards, >> -Prabath >> >> >> On Fri, Mar 4, 2011 at 8:54 AM, Dimuthu Leelarathne wrote: >> >>> Hi, >>> >>> Yes. You have a point. >>> >>> +1 for fixing it to get the configuration context. >>> >>> tx, >>> dimuthu >>> >>> >>> >>> On Fri, Mar 4, 2011 at 8:50 AM, Prabath Siriwardana wrote: >>> I have a client out side the Carbon - which uses following API UserRealm realm = WSRealmBuilder.createWSRealm(serverUrl, cookie, configContext); But this code doesn't work with the current trunk - and the API been changed.. Can you please suggest how should I replace the above..? -- Thanks & Regards, Prabath http://blog.facilelogin.com http://RampartFAQ.com ___ Carbon-dev mailing list Carbon-dev@wso2.org http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev >>> >> >> >> -- >> Thanks & Regards, >> Prabath >> >> http://blog.facilelogin.com >> http://RampartFAQ.com >> > > ___ Carbon-dev mailing list Carbon-dev@wso2.org http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev
Re: [Carbon-dev] WSRealmBuilder API changes..
Hi, The old method available was, WSRealmBuilder.createWSRealm(realmProperties, configurationContext, tenantID) TenantId is meaningless here. Why? Because shouldn't expose an API where IHC tenant can see some other tenant's data. So we cannot go back to old API. We should have two methods as follows, WSRealmBuilder.createWSRealm(realmProperties, configurationContext) WSRealmBuilder.createWSRealm(realmProperties) If super tenant wants to access other tenant's data we can discuss it (consider pros and cons) and make it available for super tenant only. Thanks, Dimuthu On Fri, Mar 4, 2011 at 9:03 AM, Prabath Siriwardana wrote: > Can we get the same old API back.. if it doesn't violate the reason it's > been removed previously..? > > Thanks & regards, > -Prabath > > > On Fri, Mar 4, 2011 at 8:54 AM, Dimuthu Leelarathne wrote: > >> Hi, >> >> Yes. You have a point. >> >> +1 for fixing it to get the configuration context. >> >> tx, >> dimuthu >> >> >> >> On Fri, Mar 4, 2011 at 8:50 AM, Prabath Siriwardana wrote: >> >>> I have a client out side the Carbon - which uses following API >>> >>> UserRealm realm = WSRealmBuilder.createWSRealm(serverUrl, cookie, >>> configContext); >>> >>> But this code doesn't work with the current trunk - and the API been >>> changed.. >>> >>> Can you please suggest how should I replace the above..? >>> >>> -- >>> Thanks & Regards, >>> Prabath >>> >>> http://blog.facilelogin.com >>> http://RampartFAQ.com >>> >>> ___ >>> Carbon-dev mailing list >>> Carbon-dev@wso2.org >>> http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev >>> >>> >> > > > -- > Thanks & Regards, > Prabath > > http://blog.facilelogin.com > http://RampartFAQ.com > ___ Carbon-dev mailing list Carbon-dev@wso2.org http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev
Re: [Carbon-dev] WSRealmBuilder API changes..
Can we get the same old API back.. if it doesn't violate the reason it's been removed previously..? Thanks & regards, -Prabath On Fri, Mar 4, 2011 at 8:54 AM, Dimuthu Leelarathne wrote: > Hi, > > Yes. You have a point. > > +1 for fixing it to get the configuration context. > > tx, > dimuthu > > > > On Fri, Mar 4, 2011 at 8:50 AM, Prabath Siriwardana wrote: > >> I have a client out side the Carbon - which uses following API >> >> UserRealm realm = WSRealmBuilder.createWSRealm(serverUrl, cookie, >> configContext); >> >> But this code doesn't work with the current trunk - and the API been >> changed.. >> >> Can you please suggest how should I replace the above..? >> >> -- >> Thanks & Regards, >> Prabath >> >> http://blog.facilelogin.com >> http://RampartFAQ.com >> >> ___ >> Carbon-dev mailing list >> Carbon-dev@wso2.org >> http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev >> >> > -- Thanks & Regards, Prabath http://blog.facilelogin.com http://RampartFAQ.com ___ Carbon-dev mailing list Carbon-dev@wso2.org http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev
Re: [Carbon-dev] WSRealmBuilder API changes..
Hi, Yes. You have a point. +1 for fixing it to get the configuration context. tx, dimuthu On Fri, Mar 4, 2011 at 8:50 AM, Prabath Siriwardana wrote: > I have a client out side the Carbon - which uses following API > > UserRealm realm = WSRealmBuilder.createWSRealm(serverUrl, cookie, > configContext); > > But this code doesn't work with the current trunk - and the API been > changed.. > > Can you please suggest how should I replace the above..? > > -- > Thanks & Regards, > Prabath > > http://blog.facilelogin.com > http://RampartFAQ.com > > ___ > Carbon-dev mailing list > Carbon-dev@wso2.org > http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev > > ___ Carbon-dev mailing list Carbon-dev@wso2.org http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev
[Carbon-dev] WSRealmBuilder API changes..
I have a client out side the Carbon - which uses following API UserRealm realm = WSRealmBuilder.createWSRealm(serverUrl, cookie, configContext); But this code doesn't work with the current trunk - and the API been changed.. Can you please suggest how should I replace the above..? -- Thanks & Regards, Prabath http://blog.facilelogin.com http://RampartFAQ.com ___ Carbon-dev mailing list Carbon-dev@wso2.org http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev