Mercurial > prosody-modules
changeset 1907:7fe7bd7b33b6
mod_cloud_notify: Allow the 'node' to be left out when disabling, clients MAY include it (and therefore leave it out)
author | Kim Alvefur <zash@zash.se> |
---|---|
date | Wed, 14 Oct 2015 13:41:03 +0200 (2015-10-14) |
parents | 5849d70bcd37 |
children | eba279ddc050 |
files | mod_cloud_notify/mod_cloud_notify.lua |
diffstat | 1 files changed, 18 insertions(+), 9 deletions(-) [+] |
line wrap: on
line diff
--- a/mod_cloud_notify/mod_cloud_notify.lua Wed Oct 14 13:20:59 2015 +0200 +++ b/mod_cloud_notify/mod_cloud_notify.lua Wed Oct 14 13:41:03 2015 +0200 @@ -13,11 +13,15 @@ local push_enabled = module:shared("push-enabled-users"); +-- http://xmpp.org/extensions/xep-0357.html#enabling module:hook("iq-set/self/"..xmlns_push..":enable", function (event) local origin, stanza = event.origin, event.stanza; - local push_jid, push_node = stanza.tags[1].attr.jid, stanza.tags[1].attr.node; - if not push_jid or not push_node then - origin.send(st.error_reply(stanza, "modify", "bad-request", "Missing jid or node")); + -- MUST contain a 'jid' attribute of the XMPP Push Service being enabled + local push_jid = stanza.tags[1].attr.jid; + -- SHOULD contain a 'node' attribute + local push_node = stanza.tags[1].attr.node; + if not push_jid then + origin.send(st.error_reply(stanza, "modify", "bad-request", "Missing jid")); return true; end local publish_options = stanza.tags[1].tags[1]; @@ -29,7 +33,7 @@ if not user_push_services then user_push_services = {}; end - user_push_services[push_jid .. "<" .. push_node] = { + user_push_services[push_jid .. "<" .. (push_node or "")] = { jid = push_jid; node = push_node; count = 0; @@ -39,16 +43,20 @@ return true; end); +-- http://xmpp.org/extensions/xep-0357.html#disabling module:hook("iq-set/self/"..xmlns_push..":disable", function (event) local origin, stanza = event.origin, event.stanza; - local push_jid, push_node = stanza.tags[1].attr.jid, stanza.tags[1].attr.node; - if not push_jid or not push_node then - origin.send(st.error_reply(stanza, "modify", "bad-request", "Missing jid or node")); + local push_jid = stanza.tags[1].attr.jid; -- MUST include a 'jid' attribute + local push_node = stanza.tags[1].attr.node; -- A 'node' attribute MAY be included + if not push_jid then + origin.send(st.error_reply(stanza, "modify", "bad-request", "Missing jid")); return true; end local user_push_services = push_enabled[origin.username]; - if user_push_services then - user_push_services[push_jid .. "<" .. push_node] = nil; + for key, push_info in pairs(user_push_services) do + if push_info.jid == push_jid and (not push_node or push_info.node == push_node) then + user_push_services[key] = nil; + end end origin.send(st.reply(stanza)); return true; @@ -62,6 +70,7 @@ { name = "last-message-body"; type = "text-single"; }; }; +-- http://xmpp.org/extensions/xep-0357.html#publishing module:hook("message/offline/handle", function(event) local origin, stanza = event.origin, event.stanza; local to = stanza.attr.to;