On Sat, May 7, 2011 at 8:32 AM, Danushka Menikkumbura <danus...@wso2.com>wrote:
> It still logs an error. Since this is a problem with bam data publishers and currently has fixed with setting proper tenant id please revert this change. It is not a nice thing to catch NullPointerException and the log messages may gone undetected. thanks, Amila. > > > On Sat, May 7, 2011 at 8:02 AM, Amila Suriarachchi <am...@wso2.com> wrote: > >> >> >> On Sat, May 7, 2011 at 6:23 AM, Danushka Menikkumbura >> <danus...@wso2.com>wrote: >> >>> Hi Amila, >>> >>> I think there was some miscommunication. Supun, correct me if I am wrong. >>> >>> Anyway I have now handled NPE in the Qpid code too. >>> >> >> If the reason what I have given is correct please revert this change. This >> may lead to such wrong invocations gone undetected. >> >> thanks, >> Amila. >> >> >>> >>> But still I wonder how the Qpid call passes a -1. See the code snippet >>> given below. This was done last week when we tested MB Stratos version. >>> >>> // Get tenant ID >>> int tenantID = 0; >>> int domainNameSeparatorIndex = >>> username.indexOf(DOMAIN_NAME_SEPARATOR); >>> if (-1 != domainNameSeparatorIndex) { // Service case >>> String domainName = >>> username.substring(domainNameSeparatorIndex + 1); >>> tenantID = >>> realmService.getTenantManager().getTenantId(domainName); >>> } >>> >>> // Get Realm >>> userRealm = realmService.getTenantUserRealm(tenantID); >>> >>> Thanks, >>> Danushka >>> >>> >>> On Sat, May 7, 2011 at 2:32 AM, Amila Jayasekara <ami...@wso2.com>wrote: >>> >>>> Hi Danushka, >>>> >>>> I did bit of debugging with the ESB pack i got from Supun and found >>>> following. >>>> >>>> The tenant id passed to DefaultRealmService.getTenantUserRealm from >>>> QpidAuthorizationPlugin.getUserRealm is -1. Thus according to >>>> DefaultRealmService.getTenantUserRealm logic the tenant id -1 does not >>>> have a special meaning and also it is not representing a normal >>>> tenant. Since RealmService cannot find the appropriate tenant when >>>> trying to retrieve Realm it gives a null pointer exception. >>>> >>>> I believe what you actually need is super tenant realm (Since this is >>>> running in stand alone mode). You need to pass 0 instead of -1. >>>> >>>> I will add a null check to tenant, before getting RealmService; to >>>> give a descriptive error message. >>>> >>>> Thanks >>>> AmilaJ >>>> >>>> >>>> On Sat, May 7, 2011 at 2:05 AM, Amila Jayasekara <ami...@wso2.com> >>>> wrote: >>>> > On Fri, May 6, 2011 at 8:12 PM, Danushka Menikkumbura < >>>> danus...@wso2.com> wrote: >>>> > AmilaJ fixed the issue in DefaultRealmService as I heard. >>>> > >>>> > Hi Danushka, >>>> > >>>> > I didnt do any modifications to DefaultRealmService with this regard. >>>> > Similar error was reported in "[Carbon-dev] QPid exception when trying >>>> > to invoke an admin service" mail thread. >>>> > But later in the same thread you informed the issue was not >>>> > reproducible with a fresh installation. >>>> > >>>> > Let me again investigate the pack i got from Supun. >>>> > >>>> > Thanks >>>> > AmilaJ >>>> > >>>> >> >>>> >> Danushka >>>> >> >>>> >> On Fri, May 6, 2011 at 5:18 PM, Tharindu Mathew <thari...@wso2.com> >>>> wrote: >>>> >>> >>>> >>> Hi, >>>> >>> We came up with this during smoke tests. Events published by ESB are >>>> not >>>> >>> being received by BAM due to this. I believe it's due to the qpid >>>> error. >>>> >>> Please have a look. >>>> >>> Stacktrace: >>>> >>> [2011-05-06 17:04:35,518] DEBUG - PublisherUtils Event payload >>>> >>> <svrusrdata:Event >>>> >>> xmlns:svrusrdata=" >>>> http://wso2.org/ns/2009/09/bam/server/user-defined/data >>>> "><svrusrdata:ServerUserDefinedData><svrusrdata:ServerName> >>>> https://10.100.0.191:9444 >>>> </svrusrdata:ServerName><svrusrdata:TenantID>0</svrusrdata:TenantID><svrusrdata:Data><svrusrdata:Key>ProxyOutMaxProcessingTime-lafProxy</svrusrdata:Key><svrusrdata:Value>1</svrusrdata:Value></svrusrdata:Data><svrusrdata:Data><svrusrdata:Key>ProxyOutAvgProcessingTime-lafProxy</svrusrdata:Key><svrusrdata:Value>0.1111111111111111</svrusrdata:Value></svrusrdata:Data><svrusrdata:Data><svrusrdata:Key>ProxyOutMinProcessingTime-lafProxy</svrusrdata:Key><svrusrdata:Value>0</svrusrdata:Value></svrusrdata:Data><svrusrdata:Data><svrusrdata:Key>ProxyOutCount-lafProxy</svrusrdata:Key><svrusrdata:Value>27</svrusrdata:Value></svrusrdata:Data><svrusrdata:Data><svrusrdata:Key>ProxyOutCumulativeCount-lafProxy</svrusrdata:Key><svrusrdata:Value>27</svrusrdata:Value></svrusrdata:Data><svrusrdata:Data><svrusrdata:Key>ProxyOutFaultCount-lafProxy</svrusrdata:Key><svrusrdata:Value>0</svrusrdata:Value></svrusrdata:Data><svrusrdata:Data><svrusrdata:Key>ProxyOutID</svrusrdata:Key><svrusrdata:Value>lafProxy</svrusrdata:Value></svrusrdata:Data></svrusrdata:ServerUserDefinedData></svrusrdata:Event> >>>> >>> [2011-05-06 17:04:35,518] DEBUG - BAMMediationStatisticsObserver >>>> >>> ComponentType.PROXYSERVICE <svrusrdata:Event >>>> >>> xmlns:svrusrdata=" >>>> http://wso2.org/ns/2009/09/bam/server/user-defined/data >>>> "><svrusrdata:ServerUserDefinedData><svrusrdata:ServerName> >>>> https://10.100.0.191:9444 >>>> </svrusrdata:ServerName><svrusrdata:TenantID>0</svrusrdata:TenantID><svrusrdata:Data><svrusrdata:Key>ProxyOutMaxProcessingTime-lafProxy</svrusrdata:Key><svrusrdata:Value>1</svrusrdata:Value></svrusrdata:Data><svrusrdata:Data><svrusrdata:Key>ProxyOutAvgProcessingTime-lafProxy</svrusrdata:Key><svrusrdata:Value>0.1111111111111111</svrusrdata:Value></svrusrdata:Data><svrusrdata:Data><svrusrdata:Key>ProxyOutMinProcessingTime-lafProxy</svrusrdata:Key><svrusrdata:Value>0</svrusrdata:Value></svrusrdata:Data><svrusrdata:Data><svrusrdata:Key>ProxyOutCount-lafProxy</svrusrdata:Key><svrusrdata:Value>27</svrusrdata:Value></svrusrdata:Data><svrusrdata:Data><svrusrdata:Key>ProxyOutCumulativeCount-lafProxy</svrusrdata:Key><svrusrdata:Value>27</svrusrdata:Value></svrusrdata:Data><svrusrdata:Data><svrusrdata:Key>ProxyOutFaultCount-lafProxy</svrusrdata:Key><svrusrdata:Value>0</svrusrdata:Value></svrusrdata:Data><svrusrdata:Data><svrusrdata:Key>ProxyOutID</svrusrdata:Key><svrusrdata:Value>lafProxy</svrusrdata:Value></svrusrdata:Data></svrusrdata:ServerUserDefinedData></svrusrdata:Event> >>>> >>> [2011-05-06 17:04:35,528] DEBUG - BAMMediationStatisticsObserver >>>> Event is >>>> >>> published<svrusrdata:Event >>>> >>> xmlns:svrusrdata=" >>>> http://wso2.org/ns/2009/09/bam/server/user-defined/data >>>> "><svrusrdata:ServerUserDefinedData><svrusrdata:ServerName> >>>> https://10.100.0.191:9444 >>>> </svrusrdata:ServerName><svrusrdata:TenantID>0</svrusrdata:TenantID><svrusrdata:Data><svrusrdata:Key>ProxyOutMaxProcessingTime-lafProxy</svrusrdata:Key><svrusrdata:Value>1</svrusrdata:Value></svrusrdata:Data><svrusrdata:Data><svrusrdata:Key>ProxyOutAvgProcessingTime-lafProxy</svrusrdata:Key><svrusrdata:Value>0.1111111111111111</svrusrdata:Value></svrusrdata:Data><svrusrdata:Data><svrusrdata:Key>ProxyOutMinProcessingTime-lafProxy</svrusrdata:Key><svrusrdata:Value>0</svrusrdata:Value></svrusrdata:Data><svrusrdata:Data><svrusrdata:Key>ProxyOutCount-lafProxy</svrusrdata:Key><svrusrdata:Value>27</svrusrdata:Value></svrusrdata:Data><svrusrdata:Data><svrusrdata:Key>ProxyOutCumulativeCount-lafProxy</svrusrdata:Key><svrusrdata:Value>27</svrusrdata:Value></svrusrdata:Data><svrusrdata:Data><svrusrdata:Key>ProxyOutFaultCount-lafProxy</svrusrdata:Key><svrusrdata:Value>0</svrusrdata:Value></svrusrdata:Data><svrusrdata:Data><svrusrdata:Key>ProxyOutID</svrusrdata:Key><svrusrdata:Value>lafProxy</svrusrdata:Value></svrusrdata:Data></svrusrdata:ServerUserDefinedData></svrusrdata:Event> >>>> >>> [2011-05-06 17:04:35,597] ERROR - DefaultRealmService >>>> >>> java.lang.NullPointerException >>>> >>> at >>>> >>> >>>> org.wso2.carbon.user.core.common.DefaultRealmService.getTenantUserRealm(DefaultRealmService.java:157) >>>> >>> at >>>> >>> >>>> org.wso2.carbon.qpid.authorization.service.qpid.QpidAuthorizationPlugin.getUserRealm(QpidAuthorizationPlugin.java:184) >>>> >>> at >>>> >>> >>>> org.wso2.carbon.qpid.authorization.service.qpid.QpidAuthorizationPlugin.authorise(QpidAuthorizationPlugin.java:124) >>>> >>> at >>>> >>> >>>> org.apache.qpid.server.security.SecurityManager$11.allowed(SecurityManager.java:390) >>>> >>> at >>>> >>> >>>> org.apache.qpid.server.security.SecurityManager.checkAllPlugins(SecurityManager.java:245) >>>> >>> at >>>> >>> >>>> org.apache.qpid.server.security.SecurityManager.authorisePublish(SecurityManager.java:386) >>>> >>> at >>>> >>> >>>> org.apache.qpid.server.transport.ServerSessionDelegate.messageTransfer(ServerSessionDelegate.java:307) >>>> >>> at >>>> >>> >>>> org.apache.qpid.server.transport.ServerSessionDelegate.messageTransfer(ServerSessionDelegate.java:96) >>>> >>> at >>>> >>> >>>> org.apache.qpid.transport.MessageTransfer.dispatch(MessageTransfer.java:108) >>>> >>> at >>>> >>> >>>> org.apache.qpid.transport.SessionDelegate.command(SessionDelegate.java:50) >>>> >>> at >>>> >>> >>>> org.apache.qpid.server.transport.ServerSessionDelegate.command(ServerSessionDelegate.java:112) >>>> >>> at >>>> >>> >>>> org.apache.qpid.server.transport.ServerSessionDelegate.command(ServerSessionDelegate.java:96) >>>> >>> at org.apache.qpid.transport.Method.delegate(Method.java:159) >>>> >>> at org.apache.qpid.transport.Session.received(Session.java:500) >>>> >>> at >>>> org.apache.qpid.transport.Connection.dispatch(Connection.java:404) >>>> >>> at >>>> >>> >>>> org.apache.qpid.transport.ConnectionDelegate.handle(ConnectionDelegate.java:64) >>>> >>> at >>>> >>> >>>> org.apache.qpid.transport.ConnectionDelegate.handle(ConnectionDelegate.java:40) >>>> >>> at >>>> >>> >>>> org.apache.qpid.transport.MethodDelegate.messageTransfer(MethodDelegate.java:113) >>>> >>> at >>>> >>> >>>> org.apache.qpid.transport.MessageTransfer.dispatch(MessageTransfer.java:108) >>>> >>> at >>>> >>> >>>> org.apache.qpid.transport.ConnectionDelegate.command(ConnectionDelegate.java:54) >>>> >>> at >>>> >>> >>>> org.apache.qpid.transport.ConnectionDelegate.command(ConnectionDelegate.java:40) >>>> >>> at org.apache.qpid.transport.Method.delegate(Method.java:159) >>>> >>> at >>>> org.apache.qpid.transport.Connection.received(Connection.java:369) >>>> >>> at >>>> >>> >>>> org.apache.qpid.server.transport.ServerConnection.received(ServerConnection.java:196) >>>> >>> at >>>> >>> >>>> org.apache.qpid.server.transport.ServerConnection.received(ServerConnection.java:53) >>>> >>> at >>>> org.apache.qpid.transport.network.Assembler.emit(Assembler.java:95) >>>> >>> at >>>> >>> >>>> org.apache.qpid.transport.network.Assembler.assemble(Assembler.java:217) >>>> >>> at >>>> org.apache.qpid.transport.network.Assembler.frame(Assembler.java:129) >>>> >>> at org.apache.qpid.transport.network.Frame.delegate(Frame.java:133) >>>> >>> at >>>> >>> >>>> org.apache.qpid.transport.network.Assembler.received(Assembler.java:100) >>>> >>> at >>>> org.apache.qpid.transport.network.Assembler.received(Assembler.java:42) >>>> >>> at >>>> >>> >>>> org.apache.qpid.transport.network.InputHandler.next(InputHandler.java:187) >>>> >>> at >>>> >>> >>>> org.apache.qpid.transport.network.InputHandler.received(InputHandler.java:103) >>>> >>> at >>>> >>> >>>> org.apache.qpid.transport.network.InputHandler.received(InputHandler.java:42) >>>> >>> at >>>> >>> >>>> org.apache.qpid.server.protocol.MultiVersionProtocolEngine.received(MultiVersionProtocolEngine.java:102) >>>> >>> at >>>> >>> >>>> org.apache.qpid.server.protocol.MultiVersionProtocolEngine.received(MultiVersionProtocolEngine.java:36) >>>> >>> at >>>> >>> >>>> org.apache.qpid.transport.network.mina.MINANetworkDriver.messageReceived(MINANetworkDriver.java:337) >>>> >>> at >>>> >>> >>>> org.apache.mina.common.support.AbstractIoFilterChain$TailFilter.messageReceived(AbstractIoFilterChain.java:703) >>>> >>> at >>>> >>> >>>> org.apache.mina.common.support.AbstractIoFilterChain.callNextMessageReceived(AbstractIoFilterChain.java:362) >>>> >>> at >>>> >>> >>>> org.apache.mina.common.support.AbstractIoFilterChain.access$1200(AbstractIoFilterChain.java:54) >>>> >>> at >>>> >>> >>>> org.apache.mina.common.support.AbstractIoFilterChain$EntryImpl$1.messageReceived(AbstractIoFilterChain.java:800) >>>> >>> at >>>> >>> >>>> org.apache.mina.filter.executor.ExecutorFilter.processEvent(ExecutorFilter.java:243) >>>> >>> at >>>> >>> >>>> org.apache.mina.filter.executor.ExecutorFilter$ProcessEventsRunnable.run(ExecutorFilter.java:305) >>>> >>> at >>>> >>> >>>> edu.emory.mathcs.backport.java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1061) >>>> >>> at >>>> >>> >>>> edu.emory.mathcs.backport.java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:575) >>>> >>> at java.lang.Thread.run(Thread.java:619) >>>> >>> [2011-05-06 17:04:35,600] WARN - QpidAuthorizationPlugin Error >>>> while >>>> >>> getting tenant user realm for user null!null >>>> >>> >>>> >>> -- >>>> >>> Regards, >>>> >>> >>>> >>> Tharindu >>>> >>> >>>> >> >>>> >> >>>> >> _______________________________________________ >>>> >> 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 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 mailing list Carbon-dev@wso2.org http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev