Re: svn commit: r791261 - /mina/sandbox/vysper/trunk/src/main/java/org/apache/vysper/xmpp/modules/extension/xep0060_pubsub/handler/PubSubCreateNodeHandler.java

2009-07-07 Thread Bernd Fondermann
mj...@apache.org wrote: Author: mjakl Date: Sun Jul 5 15:42:12 2009 New Revision: 791261 URL: http://svn.apache.org/viewvc?rev=791261view=rev Log: Don't trust the sender with the to address, use the one in the sessionContext. Now _you_ are getting overly defensive. ;-) If my mind

Re: svn commit: r791261 - /mina/sandbox/vysper/trunk/src/main/java/org/apache/vysper/xmpp/modules/extension/xep0060_pubsub/handler/PubSubCreateNodeHandler.java

2009-07-07 Thread Michael Jakl
Hi! On Tue, Jul 7, 2009 at 11:21, Bernd Fondermannbf_...@brainlounge.de wrote: Don't trust the sender with the to address, use the one in the sessionContext. Now _you_ are getting overly defensive. ;-) If my mind serves me right, the to address has been checked before (in

Re: svn commit: r791261 - /mina/sandbox/vysper/trunk/src/main/java/org/apache/vysper/xmpp/modules/extension/xep0060_pubsub/handler/PubSubCreateNodeHandler.java

2009-07-07 Thread Bernd Fondermann
Michael Jakl wrote: Hi! On Tue, Jul 7, 2009 at 11:21, Bernd Fondermannbf_...@brainlounge.de wrote: Don't trust the sender with the to address, use the one in the sessionContext. Now _you_ are getting overly defensive. ;-) If my mind serves me right, the to address has been checked before

Re: svn commit: r791261 - /mina/sandbox/vysper/trunk/src/main/java/org/apache/vysper/xmpp/modules/extension/xep0060_pubsub/handler/PubSubCreateNodeHandler.java

2009-07-07 Thread Michael Jakl
Hi! On Tue, Jul 7, 2009 at 12:06, Bernd Fondermannbf_...@brainlounge.de wrote: Michael Jakl wrote: Currently it should (why not?), but as soon as we allow components to have different domains it doesn't anymore. That's what I had in mind. What if the pubsub is addressed as

Re: svn commit: r791261 - /mina/sandbox/vysper/trunk/src/main/java/org/apache/vysper/xmpp/modules/extension/xep0060_pubsub/handler/PubSubCreateNodeHandler.java

2009-07-07 Thread Bernd Fondermann
Michael Jakl wrote: Hi! On Tue, Jul 7, 2009 at 12:06, Bernd Fondermannbf_...@brainlounge.de wrote: Michael Jakl wrote: Currently it should (why not?), but as soon as we allow components to have different domains it doesn't anymore. That's what I had in mind. What if the pubsub is addressed