NPE in DiscoExtensionsForm (#1045)

Closed

wojciech.kapcia@tigase.net opened 5 years ago

2019-06-26 16:03:27.637 [in_0-upload]      StanzaProcessor.processPacket()         SEVERE:   null when processing from=sess-man@ip-10-0-14-118.us-west-2.compute.internal, to=null, DATA=[iq to="upload.malkowscy.net" id="86186DA5-C390-4E08-BC07-1FF9F0AE2D8B" type="get" xmlns="jabber:client" from="bmalkow@malkowscy.net/BeagleIM"][query xmlns="http://jabber.org/protocol/disco#info"/][/iq], SIZE=202, XMLNS=jabber:client, PRIORITY=NORMAL, PERMISSION=ADMIN, TYPE=get
java.lang.NullPointerException
	at tigase.server.BasicComponent.getDiscoExtensionsForm(BasicComponent.java:579)
	at tigase.component.modules.impl.DiscoveryModule.prepareDiscoInfoResponse(DiscoveryModule.java:170)
	at tigase.http.upload.DiscoveryModule.prepareDiscoInfoResponse(DiscoveryModule.java:41)
	at tigase.component.modules.impl.DiscoveryModule.processDiscoInfo(DiscoveryModule.java:150)
	at tigase.component.modules.impl.DiscoveryModule.process(DiscoveryModule.java:116)
	at tigase.component.modules.StanzaProcessor.process(StanzaProcessor.java:164)
	at tigase.component.modules.StanzaProcessor.processPacket(StanzaProcessor.java:85)
	at tigase.component.AbstractKernelBasedComponent.processPacket(AbstractKernelBasedComponent.java:81)
	at tigase.server.AbstractMessageReceiver$QueueListener.run(AbstractMessageReceiver.java:1319)

2019-06-26 16:03:43.815 [in_0-upload]      StanzaProcessor.processPacket()         SEVERE:   null when processing from=sess-man@ip-10-0-25-189.us-west-2.compute.internal, to=null, DATA=[iq id="a10eba" to="upload.tigase.org" type="get" xmlns="jabber:client" from="wojtek@tigase.org/atlantis/city/psi+"][query xmlns="http://jabber.org/protocol/disco#info"/][/iq], SIZE=175, XMLNS=jabber:client, PRIORITY=NORMAL, PERMISSION=ADMIN, TYPE=get
java.lang.NullPointerException
	at tigase.server.BasicComponent.getDiscoExtensionsForm(BasicComponent.java:579)
	at tigase.component.modules.impl.DiscoveryModule.prepareDiscoInfoResponse(DiscoveryModule.java:170)
	at tigase.http.upload.DiscoveryModule.prepareDiscoInfoResponse(DiscoveryModule.java:41)
	at tigase.component.modules.impl.DiscoveryModule.processDiscoInfo(DiscoveryModule.java:150)
	at tigase.component.modules.impl.DiscoveryModule.process(DiscoveryModule.java:116)
	at tigase.component.modules.StanzaProcessor.process(StanzaProcessor.java:164)
	at tigase.component.modules.StanzaProcessor.processPacket(StanzaProcessor.java:85)
	at tigase.component.AbstractKernelBasedComponent.processPacket(AbstractKernelBasedComponent.java:81)
	at tigase.server.AbstractMessageReceiver$QueueListener.run(AbstractMessageReceiver.java:1319)

Andrzej Wójcik (Tigase) commented 5 years ago

@wojtek The issue here is that I've expected that VHost domain is passed as a parameter to this method while there can be a subdomain in case of a component, i.e upload.example.com for vhost example.com. I've made proper adjustments and now it should work just fine.

Type	Bug
Priority	Normal
Assignee	wojciech.kapcia@tigase.net
Version	tigase-server-8.1.0
Spent time	1h 15m

Issue Votes (0)

Watchers (0)

Reference

tigase/_server/server-core#1045