Mercurial > prosody-modules
comparison mod_adhoc/adhoc/mod_adhoc.lua @ 93:611d16867410
mod_adhoc: Check for global and host admins
author | Florian Zeitz <florob@babelmonkeys.de> |
---|---|
date | Sat, 14 Nov 2009 18:44:54 +0100 |
parents | 59f490390528 |
children | 9b63fd1196c0 |
comparison
equal
deleted
inserted
replaced
92:7dad958aad15 | 93:611d16867410 |
---|---|
10 | 10 |
11 module:add_feature("http://jabber.org/protocol/commands"); | 11 module:add_feature("http://jabber.org/protocol/commands"); |
12 | 12 |
13 module:hook("iq/host/http://jabber.org/protocol/disco#items:query", function (event) | 13 module:hook("iq/host/http://jabber.org/protocol/disco#items:query", function (event) |
14 local origin, stanza = event.origin, event.stanza; | 14 local origin, stanza = event.origin, event.stanza; |
15 local privileged = is_admin(event.stanza.attr.from); | 15 local privileged = is_admin(event.stanza.attr.from) or is_admin(stanza.attr.from, stanza.attr.to); -- TODO: Is this correct, or should is_admin be changed? |
16 if stanza.attr.type == "get" and stanza.tags[1].attr.node and stanza.tags[1].attr.node == "http://jabber.org/protocol/commands" then | 16 if stanza.attr.type == "get" and stanza.tags[1].attr.node and stanza.tags[1].attr.node == "http://jabber.org/protocol/commands" then |
17 reply = st.reply(stanza); | 17 reply = st.reply(stanza); |
18 reply:tag("query", {xmlns="http://jabber.org/protocol/disco#items", node="http://jabber.org/protocol/commands"}) | 18 reply:tag("query", {xmlns="http://jabber.org/protocol/disco#items", node="http://jabber.org/protocol/commands"}) |
19 for i = 1, #commands do | 19 for i = 1, #commands do |
20 -- module:log("info", "adding command %s", commands[i].name); | 20 -- module:log("info", "adding command %s", commands[i].name); |
30 | 30 |
31 module:hook("iq/host", function (event) | 31 module:hook("iq/host", function (event) |
32 local origin, stanza = event.origin, event.stanza; | 32 local origin, stanza = event.origin, event.stanza; |
33 if stanza.attr.type == "set" and stanza.tags[1] and stanza.tags[1].name == "command" then | 33 if stanza.attr.type == "set" and stanza.tags[1] and stanza.tags[1].name == "command" then |
34 local node = stanza.tags[1].attr.node | 34 local node = stanza.tags[1].attr.node |
35 for i = 1, #commands do | 35 local privileged = is_admin(event.stanza.attr.from) or is_admin(stanza.attr.from, stanza.attr.to); -- TODO: Is this correct, or should is_admin be changed? |
36 if commands[i].node == node then | 36 for i = 1, #commands do |
37 -- check whether user has permission to execute this command first | 37 if commands[i].node == node then |
38 if commands[i].permission == "admin" and not is_admin(stanza.attr.from) then | 38 -- check whether user has permission to execute this command first |
39 origin.send(st.error_reply(stanza, "auth", "forbidden", "You don't have permission to execute this command"):up() | 39 if commands[i].permission == "admin" and not privileged then |
40 :add_child(commands[i]:cmdtag("canceled") | 40 origin.send(st.error_reply(stanza, "auth", "forbidden", "You don't have permission to execute this command"):up() |
41 :tag("note", {type="error"}):text("You don't have permission to execute this command"))); | 41 :add_child(commands[i]:cmdtag("canceled") |
42 return true | 42 :tag("note", {type="error"}):text("You don't have permission to execute this command"))); |
43 end | 43 return true |
44 -- User has permission now execute the command | |
45 return commands[i].handler(commands[i], origin, stanza); | |
46 end | 44 end |
45 -- User has permission now execute the command | |
46 return commands[i].handler(commands[i], origin, stanza); | |
47 end | 47 end |
48 end | |
48 end | 49 end |
49 end, 500); | 50 end, 500); |
50 | 51 |
51 module:hook("item-added/adhoc", function (event) | 52 module:hook("item-added/adhoc", function (event) |
52 commands[ # commands + 1] = event.item; | 53 commands[ # commands + 1] = event.item; |