mirrors/prosody
1
0
Fork 0
mirror of https://github.com/bjc/prosody.git synced 2025-04-03 21:27:38 +03:00
Code Issues Projects Releases Packages Wiki Activity Actions

mod_pubsub: Include <pubsub> with unsubscribe reply

Browse source 
XEP-0060 6.2.2

This is a MAY but it makes it nice and symmetric with the subscription
response. Reduces the need to remember which node you unsubscribed from.

Explicit > implicit etc.
This commit is contained in:
Kim Alvefur 2021-03-15 19:04:08 +01:00
parent 6bae44fea8
commit 90e4a09661
2 changed files with 12 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

8
plugins/mod_pubsub/pubsub.lib.lua
View file

@ -517,7 +517,13 @@ function handlers.set_unsubscribe(origin, stanza, unsubscribe, service)
local ok, ret = service:remove_subscription(node, stanza.attr.from, jid);
local reply;
if ok then
reply = st.reply(stanza);
reply = st.reply(stanza)
:tag("pubsub", { xmlns = xmlns_pubsub })
:tag("subscription", {
node = node,
jid = jid,
subscription = "none"
}):up();
else
reply = pubsub_error_reply(stanza, ret);
end

6
spec/scansion/pubsub_advanced.scs
View file

@ -150,7 +150,11 @@ Juliet sends:
</iq>
Juliet receives:
<iq type="result" id='unsub1'/>
<iq type="result" id='unsub1'>
<pubsub xmlns='http://jabber.org/protocol/pubsub'>
<subscription jid="${Juliet's full JID}" node='princely_musings' subscription='none'/>
</pubsub>
</iq>
Balthasar sends:
<iq type="set" to="pubsub.localhost" id='del1'>