From: Matthew Wild <mwild1@gmail.com>
Date: Fri, 18 Mar 2011 10:32:55 +0000 (+0000)
Subject: mod_pubsub: Remove checks for whether the JID being (un)subscribed is the actor's... 
X-Git-Url: https://git.enpas.org/?a=commitdiff_plain;h=c66cc52a698c80e695669910c338fca24a15fd28;p=prosody.git

mod_pubsub: Remove checks for whether the JID being (un)subscribed is the actor's own - it's possible to (un)subscribe another JID if you have the (un)subscribe_other capability, which is already checked in util.pubsub (thanks Andrew)
---

diff --git a/plugins/mod_pubsub.lua b/plugins/mod_pubsub.lua
index b8f7a882..465bcb00 100644
--- a/plugins/mod_pubsub.lua
+++ b/plugins/mod_pubsub.lua
@@ -108,9 +108,6 @@ end
 
 function handlers.set_subscribe(origin, stanza, subscribe)
 	local node, jid = subscribe.attr.node, subscribe.attr.jid;
-	if jid_bare(jid) ~= jid_bare(stanza.attr.from) then
-		return origin.send(pubsub_error_reply(stanza, "invalid-jid"));
-	end
 	local ok, ret = service:add_subscription(node, stanza.attr.from, jid);
 	local reply;
 	if ok then
@@ -129,9 +126,6 @@ end
 
 function handlers.set_unsubscribe(origin, stanza, unsubscribe)
 	local node, jid = unsubscribe.attr.node, unsubscribe.attr.jid;
-	if jid_bare(jid) ~= jid_bare(stanza.attr.from) then
-		return origin.send(pubsub_error_reply(stanza, "invalid-jid"));
-	end
 	local ok, ret = service:remove_subscription(node, stanza.attr.from, jid);
 	local reply;
 	if ok then