MUC: Prevent visitors from broadcasting messages.
[prosody.git] / plugins / muc / muc.lib.lua
index 6d74c7ba9613ecddf4599918470081d0c23c4633..7907c0c3ea54674b6e70c28fffdff72e6eff34ac 100644 (file)
@@ -11,6 +11,7 @@ local datetime = require "util.datetime";
 
 local jid_split = require "util.jid".split;
 local jid_bare = require "util.jid".bare;
+local jid_prep = require "util.jid".prep;
 local st = require "util.stanza";
 local log = require "util.logger".init("mod_muc");
 local multitable_new = require "util.multitable".new;
@@ -44,7 +45,7 @@ local function filter_xmlns_from_stanza(stanza, filters)
 end
 local presence_filters = {["http://jabber.org/protocol/muc"]=true;["http://jabber.org/protocol/muc#user"]=true};
 local function get_filtered_presence(stanza)
-       return filter_xmlns_from_stanza(st.clone(stanza), presence_filters);
+       return filter_xmlns_from_stanza(st.clone(stanza):reset(), presence_filters);
 end
 local kickable_error_conditions = {
        ["gone"] = true;
@@ -56,19 +57,24 @@ local kickable_error_conditions = {
        ["remote-server-not-found"] = true;
        ["remote-server-timeout"] = true;
        ["service-unavailable"] = true;
+       ["malformed error"] = true;
 };
-local function get_kickable_error(stanza)
+local function get_error_condition(stanza)
        for _, tag in ipairs(stanza.tags) do
-               if tag.name == "error" and tag.attr.xmlns == "jabber:client" then
+               if tag.name == "error" and (not(tag.attr.xmlns) or tag.attr.xmlns == "jabber:client") then
                        for _, cond in ipairs(tag.tags) do
                                if cond.attr.xmlns == "urn:ietf:params:xml:ns:xmpp-stanzas" then
-                                       return kickable_error_conditions[cond.name] and cond.name;
+                                       return cond.name;
                                end
                        end
-                       return true; -- malformed error message
+                       return "malformed error";
                end
        end
-       return true; -- malformed error message
+       return "malformed error";
+end
+local function is_kickable_error(stanza)
+       local cond = get_error_condition(stanza);
+       return kickable_error_conditions[cond] and cond;
 end
 local function getUsingPath(stanza, path, getText)
        local tag = stanza;
@@ -105,47 +111,38 @@ function room_mt:get_default_role(affiliation)
        end
 end
 
-function room_mt:broadcast_presence(stanza, code, nick)
+function room_mt:broadcast_presence(stanza, sid, code, nick)
        stanza = get_filtered_presence(stanza);
-       local data = self._occupants[stanza.attr.from];
+       local occupant = self._occupants[stanza.attr.from];
        stanza:tag("x", {xmlns='http://jabber.org/protocol/muc#user'})
-               :tag("item", {affiliation=data.affiliation, role=data.role, nick=nick}):up();
+               :tag("item", {affiliation=occupant.affiliation or "none", role=occupant.role or "none", nick=nick}):up();
        if code then
                stanza:tag("status", {code=code}):up();
        end
-       local me;
-       for occupant, o_data in pairs(self._occupants) do
-               if occupant ~= stanza.attr.from then
-                       for jid in pairs(o_data.sessions) do
-                               stanza.attr.to = jid;
-                               self:route_stanza(stanza);
-                       end
-               else
-                       me = o_data;
-               end
-       end
+       self:broadcast_except_nick(stanza, stanza.attr.from);
+       local me = self._occupants[stanza.attr.from];
        if me then
                stanza:tag("status", {code='110'});
-               for jid in pairs(me.sessions) do
-                       stanza.attr.to = jid;
-                       self:route_stanza(stanza);
-               end
+               stanza.attr.to = sid;
+               self:_route_stanza(stanza);
        end
 end
 function room_mt:broadcast_message(stanza, historic)
+       local to = stanza.attr.to;
        for occupant, o_data in pairs(self._occupants) do
                for jid in pairs(o_data.sessions) do
                        stanza.attr.to = jid;
-                       self:route_stanza(stanza);
+                       self:_route_stanza(stanza);
                end
        end
+       stanza.attr.to = to;
        if historic then -- add to history
                local history = self._data['history'];
                if not history then history = {}; self._data['history'] = history; end
-               -- stanza = st.clone(stanza);
+               stanza = st.clone(stanza);
                stanza:tag("delay", {xmlns = "urn:xmpp:delay", from = muc_domain, stamp = datetime.datetime()}):up(); -- XEP-0203
                stanza:tag("x", {xmlns = "jabber:x:delay", from = muc_domain, stamp = datetime.legacy()}):up(); -- XEP-0091 (deprecated)
-               t_insert(history, st.clone(st.preserialize(stanza)));
+               t_insert(history, st.preserialize(stanza));
                while #history > history_length do t_remove(history, 1) end
        end
 end
@@ -154,7 +151,7 @@ function room_mt:broadcast_except_nick(stanza, nick)
                if rnick ~= nick then
                        for jid in pairs(occupant.sessions) do
                                stanza.attr.to = jid;
-                               self:route_stanza(stanza);
+                               self:_route_stanza(stanza);
                        end
                end
        end
@@ -167,8 +164,8 @@ function room_mt:send_occupant_list(to)
                        local pres = get_filtered_presence(o_data.sessions[o_data.jid]);
                        pres.attr.to, pres.attr.from = to, occupant;
                        pres:tag("x", {xmlns='http://jabber.org/protocol/muc#user'})
-                               :tag("item", {affiliation=o_data.affiliation, role=o_data.role}):up();
-                       self:route_stanza(pres);
+                               :tag("item", {affiliation=o_data.affiliation or "none", role=o_data.role or "none"}):up();
+                       self:_route_stanza(pres);
                end
        end
 end
@@ -178,19 +175,25 @@ function room_mt:send_history(to)
                for _, msg in ipairs(history) do
                        msg = st.deserialize(msg);
                        msg.attr.to=to;
-                       self:route_stanza(msg);
+                       self:_route_stanza(msg);
                end
        end
        if self._data['subject'] then
-               self:route_stanza(st.message({type='groupchat', from=self.jid, to=to}):tag("subject"):text(self._data['subject']));
+               self:_route_stanza(st.message({type='groupchat', from=self.jid, to=to}):tag("subject"):text(self._data['subject']));
        end
 end
 
 local function room_get_disco_info(self, stanza)
-       return st.reply(stanza):query("http://jabber.org/protocol/disco#info"):tag("identity", {category="conference", type="text"});
+       return st.reply(stanza):query("http://jabber.org/protocol/disco#info")
+               :tag("identity", {category="conference", type="text"}):up()
+               :tag("feature", {var="http://jabber.org/protocol/muc"});
 end
 local function room_get_disco_items(self, stanza)
-       return st.reply(stanza):query("http://jabber.org/protocol/disco#items");
+       local reply = st.reply(stanza):query("http://jabber.org/protocol/disco#items");
+       for room_jid in pairs(self._occupants) do
+               reply:tag("item", {jid = room_jid, name = room_jid:match("/(.*)")}):up();
+       end
+       return reply;
 end
 function room_mt:set_subject(current_nick, subject)
        -- TODO check nick's authority
@@ -217,15 +220,35 @@ function room_mt:handle_to_occupant(origin, stanza) -- PM, vCards, etc
                        if current_nick then
                                log("debug", "kicking %s from %s", current_nick, room);
                                self:handle_to_occupant(origin, st.presence({type='unavailable', from=from, to=to})
-                                       :tag('status'):text('This participant is kicked from the room because he sent an error presence')); -- send unavailable
+                                       :tag('status'):text('Kicked: '..get_error_condition(stanza))); -- send unavailable
                        end
                elseif type == "unavailable" then -- unavailable
                        if current_nick then
                                log("debug", "%s leaving %s", current_nick, room);
-                               local data = self._occupants[current_nick];
-                               data.role = 'none';
-                               self:broadcast_presence(pr);
-                               self._occupants[current_nick] = nil;
+                               local occupant = self._occupants[current_nick];
+                               local new_jid = next(occupant.sessions);
+                               if new_jid == from then new_jid = next(occupant.sessions, new_jid); end
+                               if new_jid then
+                                       local jid = occupant.jid;
+                                       occupant.jid = new_jid;
+                                       occupant.sessions[from] = nil;
+                                       pr.attr.to = from;
+                                       pr:tag("x", {xmlns='http://jabber.org/protocol/muc#user'})
+                                               :tag("item", {affiliation=occupant.affiliation or "none", role='none'}):up()
+                                               :tag("status", {code='110'});
+                                       self:_route_stanza(pr);
+                                       if jid ~= new_jid then
+                                               pr = st.clone(occupant.sessions[new_jid])
+                                                       :tag("x", {xmlns='http://jabber.org/protocol/muc#user'})
+                                                       :tag("item", {affiliation=occupant.affiliation or "none", role=occupant.role or "none"});
+                                               pr.attr.from = current_nick;
+                                               self:broadcast_except_nick(pr, current_nick);
+                                       end
+                               else
+                                       occupant.role = 'none';
+                                       self:broadcast_presence(pr, from);
+                                       self._occupants[current_nick] = nil;
+                               end
                                self._jid_nick[from] = nil;
                        end
                elseif not type then -- available
@@ -234,24 +257,28 @@ function room_mt:handle_to_occupant(origin, stanza) -- PM, vCards, etc
                                        if current_nick == to then -- simple presence
                                                log("debug", "%s broadcasted presence", current_nick);
                                                self._occupants[current_nick].sessions[from] = pr;
-                                               self:broadcast_presence(pr);
+                                               self:broadcast_presence(pr, from);
                                        else -- change nick
-                                               if self._occupants[to] then
+                                               local occupant = self._occupants[current_nick];
+                                               local is_multisession = next(occupant.sessions, next(occupant.sessions));
+                                               if self._occupants[to] or is_multisession then
                                                        log("debug", "%s couldn't change nick", current_nick);
-                                                       origin.send(st.error_reply(stanza, "cancel", "conflict"):tag("x", {xmlns = "http://jabber.org/protocol/muc"}));
+                                                       local reply = st.error_reply(stanza, "cancel", "conflict"):up();
+                                                       reply.tags[1].attr.code = "409";
+                                                       origin.send(reply:tag("x", {xmlns = "http://jabber.org/protocol/muc"}));
                                                else
                                                        local data = self._occupants[current_nick];
                                                        local to_nick = select(3, jid_split(to));
                                                        if to_nick then
                                                                log("debug", "%s (%s) changing nick to %s", current_nick, data.jid, to);
                                                                local p = st.presence({type='unavailable', from=current_nick});
-                                                               self:broadcast_presence(p, '303', to_nick);
+                                                               self:broadcast_presence(p, from, '303', to_nick);
                                                                self._occupants[current_nick] = nil;
                                                                self._occupants[to] = data;
                                                                self._jid_nick[from] = to;
                                                                pr.attr.from = to;
                                                                self._occupants[to].sessions[from] = pr;
-                                                               self:broadcast_presence(pr);
+                                                               self:broadcast_presence(pr, from);
                                                        else
                                                                --TODO malformed-jid
                                                        end
@@ -265,12 +292,18 @@ function room_mt:handle_to_occupant(origin, stanza) -- PM, vCards, etc
                                --end
                        else -- enter room
                                local new_nick = to;
+                               local is_merge;
                                if self._occupants[to] then
-                                       new_nick = nil;
+                                       if jid_bare(from) ~= jid_bare(self._occupants[to].jid) then
+                                               new_nick = nil;
+                                       end
+                                       is_merge = true;
                                end
                                if not new_nick then
                                        log("debug", "%s couldn't join due to nick conflict: %s", from, to);
-                                       origin.send(st.error_reply(stanza, "cancel", "conflict"):tag("x", {xmlns = "http://jabber.org/protocol/muc"}));
+                                       local reply = st.error_reply(stanza, "cancel", "conflict"):up();
+                                       reply.tags[1].attr.code = "409";
+                                       origin.send(reply:tag("x", {xmlns = "http://jabber.org/protocol/muc"}));
                                else
                                        log("debug", "%s joining as %s", from, to);
                                        if not next(self._affiliations) then -- new room, no owners
@@ -279,19 +312,34 @@ function room_mt:handle_to_occupant(origin, stanza) -- PM, vCards, etc
                                        local affiliation = self:get_affiliation(from);
                                        local role = self:get_default_role(affiliation)
                                        if role then -- new occupant
-                                               self._occupants[to] = {affiliation=affiliation, role=role, jid=from, sessions={[from]=get_filtered_presence(stanza)}};
+                                               if not is_merge then
+                                                       self._occupants[to] = {affiliation=affiliation, role=role, jid=from, sessions={[from]=get_filtered_presence(stanza)}};
+                                               else
+                                                       self._occupants[to].sessions[from] = get_filtered_presence(stanza);
+                                               end
                                                self._jid_nick[from] = to;
                                                self:send_occupant_list(from);
                                                pr.attr.from = to;
-                                               self:broadcast_presence(pr);
+                                               if not is_merge then
+                                                       self:broadcast_presence(pr, from);
+                                               else
+                                                       pr.attr.to = from;
+                                                       self:_route_stanza(pr:tag("x", {xmlns='http://jabber.org/protocol/muc#user'})
+                                                               :tag("item", {affiliation=affiliation or "none", role=role or "none"}):up()
+                                                               :tag("status", {code='110'}));
+                                               end
                                                self:send_history(from);
                                        else -- banned
-                                               origin.send(st.error_reply(stanza, "auth", "forbidden"):tag("x", {xmlns = "http://jabber.org/protocol/muc"}));
+                                               local reply = st.error_reply(stanza, "auth", "forbidden"):up();
+                                               reply.tags[1].attr.code = "403";
+                                               origin.send(reply:tag("x", {xmlns = "http://jabber.org/protocol/muc"}));
                                        end
                                end
                        end
                elseif type ~= 'result' then -- bad type
-                       origin.send(st.error_reply(stanza, "modify", "bad-request")); -- FIXME correct error?
+                       if type ~= 'visible' and type ~= 'invisible' then -- COMPAT ejabberd can broadcast or forward XEP-0018 presences
+                               origin.send(st.error_reply(stanza, "modify", "bad-request")); -- FIXME correct error?
+                       end
                end
        elseif not current_nick then -- not in room
                if type == "error" or type == "result" then
@@ -308,7 +356,7 @@ function room_mt:handle_to_occupant(origin, stanza) -- PM, vCards, etc
                                end
                                if _to then
                                        stanza.attr.to, stanza.attr.from, stanza.attr.id = _to, self._jid_nick[_nick], _id;
-                                       self:route_stanza(stanza);
+                                       self:_route_stanza(stanza);
                                        stanza.attr.to, stanza.attr.from, stanza.attr.id = to, from, id;
                                end
                        end
@@ -317,6 +365,10 @@ function room_mt:handle_to_occupant(origin, stanza) -- PM, vCards, etc
                end
        elseif stanza.name == "message" and type == "groupchat" then -- groupchat messages not allowed in PM
                origin.send(st.error_reply(stanza, "modify", "bad-request"));
+       elseif current_nick and stanza.name == "message" and type == "error" and is_kickable_error(stanza) then
+               log("debug", "%s kicked from %s for sending an error message", current_nick, self.jid);
+               self:handle_to_occupant(origin, st.presence({type='unavailable', from=stanza.attr.from, to=stanza.attr.to})
+                       :tag('status'):text('Kicked: '..get_error_condition(stanza))); -- send unavailable
        else -- private stanza
                local o_data = self._occupants[to];
                if o_data then
@@ -329,7 +381,7 @@ function room_mt:handle_to_occupant(origin, stanza) -- PM, vCards, etc
                                stanza.attr.to = bare;
                                stanza.attr.id = base64.encode(jid.."\0"..id.."\0"..md5(from));
                        end
-                       self:route_stanza(stanza);
+                       self:_route_stanza(stanza);
                        stanza.attr.to, stanza.attr.from, stanza.attr.id = to, from, id;
                elseif type ~= "error" and type ~= "result" then -- recipient not in room
                        origin.send(st.error_reply(stanza, "cancel", "item-not-found", "Recipient not in room"));
@@ -401,15 +453,23 @@ function room_mt:handle_to_room(origin, stanza) -- presence changes and groupcha
                        if item and item.name == "item" then
                                if type == "set" then
                                        local callback = function() origin.send(st.reply(stanza)); end
+                                       if item.attr.jid then -- Validate provided JID
+                                               item.attr.jid = jid_prep(item.attr.jid);
+                                               if not item.attr.jid then
+                                                       origin.send(st.error_reply(stanza, "modify", "jid-malformed"));
+                                                       return;
+                                               end
+                                       end
                                        if not item.attr.jid and item.attr.nick then -- COMPAT Workaround for Miranda sending 'nick' instead of 'jid' when changing affiliation
                                                local occupant = self._occupants[self.jid.."/"..item.attr.nick];
                                                if occupant then item.attr.jid = occupant.jid; end
                                        end
+                                       local reason = item.tags[1] and item.tags[1].name == "reason" and #item.tags[1] == 1 and item.tags[1][1];
                                        if item.attr.affiliation and item.attr.jid and not item.attr.role then
-                                               local success, errtype, err = self:set_affiliation(actor, item.attr.jid, item.attr.affiliation, callback);
+                                               local success, errtype, err = self:set_affiliation(actor, item.attr.jid, item.attr.affiliation, callback, reason);
                                                if not success then origin.send(st.error_reply(stanza, errtype, err)); end
                                        elseif item.attr.role and item.attr.nick and not item.attr.affiliation then
-                                               local success, errtype, err = self:set_role(actor, self.jid.."/"..item.attr.nick, item.attr.role, callback);
+                                               local success, errtype, err = self:set_role(actor, self.jid.."/"..item.attr.nick, item.attr.role, callback, reason);
                                                if not success then origin.send(st.error_reply(stanza, errtype, err)); end
                                        else
                                                origin.send(st.error_reply(stanza, "cancel", "bad-request"));
@@ -459,8 +519,11 @@ function room_mt:handle_to_room(origin, stanza) -- presence changes and groupcha
                local from, to = stanza.attr.from, stanza.attr.to;
                local room = jid_bare(to);
                local current_nick = self._jid_nick[from];
-               if not current_nick then -- not in room
+               local occupant = self._occupants[current_nick];
+               if not occupant then -- not in room
                        origin.send(st.error_reply(stanza, "cancel", "not-acceptable"));
+               elseif occupant.role == "visitor" then
+                       origin.send(st.error_reply(stanza, "cancel", "forbidden"));
                else
                        local from = stanza.attr.from;
                        stanza.attr.from = current_nick;
@@ -470,12 +533,13 @@ function room_mt:handle_to_room(origin, stanza) -- presence changes and groupcha
                        else
                                self:broadcast_message(stanza, true);
                        end
+                       stanza.attr.from = from;
                end
-       elseif stanza.name == "message" and type == "error" and get_kickable_error(stanza) then
+       elseif stanza.name == "message" and type == "error" and is_kickable_error(stanza) then
                local current_nick = self._jid_nick[stanza.attr.from];
                log("debug", "%s kicked from %s for sending an error message", current_nick, self.jid);
                self:handle_to_occupant(origin, st.presence({type='unavailable', from=stanza.attr.from, to=stanza.attr.to})
-                       :tag('status'):text('This participant is kicked from the room because he sent an error message to another occupant')); -- send unavailable
+                       :tag('status'):text('Kicked: '..get_error_condition(stanza))); -- send unavailable
        elseif stanza.name == "presence" then -- hack - some buggy clients send presence updates to the room rather than their nick
                local to = stanza.attr.to;
                local current_nick = self._jid_nick[stanza.attr.from];
@@ -487,15 +551,33 @@ function room_mt:handle_to_room(origin, stanza) -- presence changes and groupcha
                        origin.send(st.error_reply(stanza, "cancel", "service-unavailable"));
                end
        elseif stanza.name == "message" and not stanza.attr.type and #stanza.tags == 1 and self._jid_nick[stanza.attr.from]
-               and stanza.tags[1].name == "x" and stanza.tags[1].attr.xmlns == "http://jabber.org/protocol/muc#user" and #stanza.tags[1].tags == 1
-               and stanza.tags[1].tags[1].name == "invite" and stanza.tags[1].tags[1].attr.to then
-               local _from, _to = stanza.attr.from, stanza.attr.to;
-               local _invitee = stanza.tags[1].tags[1].attr.to;
-               stanza.attr.from, stanza.attr.to = _to, _invitee;
-               stanza.tags[1].tags[1].attr.from, stanza.tags[1].tags[1].attr.to = _from, nil;
-               self:route_stanza(stanza);
-               stanza.tags[1].tags[1].attr.from, stanza.tags[1].tags[1].attr.to = nil, _invitee;
-               stanza.attr.from, stanza.attr.to = _from, _to;
+               and stanza.tags[1].name == "x" and stanza.tags[1].attr.xmlns == "http://jabber.org/protocol/muc#user" then
+               local x = stanza.tags[1];
+               local payload = (#x.tags == 1 and x.tags[1]);
+               if payload and payload.name == "invite" and payload.attr.to then
+                       local _from, _to = stanza.attr.from, stanza.attr.to;
+                       local _invitee = jid_prep(payload.attr.to);
+                       if _invitee then
+                               local _reason = payload.tags[1] and payload.tags[1].name == 'reason' and #payload.tags[1].tags == 0 and payload.tags[1][1];
+                               local invite = st.message({from = _to, to = _invitee, id = stanza.attr.id})
+                                       :tag('x', {xmlns='http://jabber.org/protocol/muc#user'})
+                                               :tag('invite', {from=_from})
+                                                       :tag('reason'):text(_reason or ""):up()
+                                               :up()
+                                       :up()
+                                       :tag('x', {xmlns="jabber:x:conference", jid=_to}) -- COMPAT: Some older clients expect this
+                                               :text(_reason or "")
+                                       :up()
+                                       :tag('body') -- Add a plain message for clients which don't support invites
+                                               :text(_from..' invited you to the room '.._to..(_reason and (' ('.._reason..')') or ""))
+                                       :up();
+                               self:_route_stanza(invite);
+                       else
+                               origin.send(st.error_reply(stanza, "cancel", "jid-malformed"));
+                       end
+               else
+                       origin.send(st.error_reply(stanza, "cancel", "bad-request"));
+               end
        else
                if type == "error" or type == "result" then return; end
                origin.send(st.error_reply(stanza, "cancel", "service-unavailable"));
@@ -520,7 +602,7 @@ function room_mt:get_affiliation(jid)
        if not result and self._affiliations[host] == "outcast" then result = "outcast"; end -- host banned
        return result;
 end
-function room_mt:set_affiliation(actor, jid, affiliation, callback)
+function room_mt:set_affiliation(actor, jid, affiliation, callback, reason)
        jid = jid_bare(jid);
        if affiliation == "none" then affiliation = nil; end
        if affiliation and affiliation ~= "outcast" and affiliation ~= "owner" and affiliation ~= "admin" and affiliation ~= "member" then
@@ -532,7 +614,9 @@ function room_mt:set_affiliation(actor, jid, affiliation, callback)
        local role = self:get_default_role(affiliation);
        local p = st.presence()
                :tag("x", {xmlns = "http://jabber.org/protocol/muc#user"})
-                       :tag("item", {affiliation=affiliation or "none", role=role or "none"}):up();
+                       :tag("item", {affiliation=affiliation or "none", role=role or "none"})
+                               :tag("reason"):text(reason or ""):up()
+                       :up();
        local x = p.tags[1];
        local item = x.tags[1];
        if not role then -- getting kicked
@@ -556,7 +640,7 @@ function room_mt:set_affiliation(actor, jid, affiliation, callback)
                        for jid in pairs(occupant.sessions) do -- remove for all sessions of the nick
                                if not role then self._jid_nick[jid] = nil; end
                                p.attr.to = jid;
-                               self:route_stanza(p);
+                               self:_route_stanza(p);
                        end
                end
        end
@@ -573,7 +657,7 @@ function room_mt:get_role(nick)
        local session = self._occupants[nick];
        return session and session.role or nil;
 end
-function room_mt:set_role(actor, nick, role, callback)
+function room_mt:set_role(actor, nick, role, callback, reason)
        if role == "none" then role = nil; end
        if role and role ~= "moderator" and role ~= "participant" and role ~= "visitor" then return nil, "modify", "not-acceptable"; end
        if self:get_affiliation(actor) ~= "owner" then return nil, "cancel", "not-allowed"; end
@@ -582,7 +666,9 @@ function room_mt:set_role(actor, nick, role, callback)
        if occupant.affiliation == "owner" or occupant.affiliation == "admin" then return nil, "cancel", "not-allowed"; end
        local p = st.presence({from = nick})
                :tag("x", {xmlns = "http://jabber.org/protocol/muc#user"})
-                       :tag("item", {affiliation=occupant.affiliation or "none", nick=nick, role=role or "none"}):up();
+                       :tag("item", {affiliation=occupant.affiliation or "none", nick=nick, role=role or "none"})
+                               :tag("reason"):text(reason or ""):up()
+                       :up();
        if not role then -- kick
                p.attr.type = "unavailable";
                self._occupants[nick] = nil;
@@ -595,13 +681,51 @@ function room_mt:set_role(actor, nick, role, callback)
        end
        for jid in pairs(occupant.sessions) do -- send to all sessions of the nick
                p.attr.to = jid;
-               self:route_stanza(p);
+               self:_route_stanza(p);
        end
        if callback then callback(); end
        self:broadcast_except_nick(p, nick);
        return true;
 end
 
+function room_mt:_route_stanza(stanza)
+       local muc_child;
+       local to_occupant = self._occupants[self._jid_nick[stanza.attr.to]];
+       local from_occupant = self._occupants[stanza.attr.from];
+       if stanza.name == "presence" then
+               if to_occupant and from_occupant then
+                       if to_occupant.role == "moderator" or jid_bare(to_occupant.jid) == jid_bare(from_occupant.jid) then
+                               for i=#stanza.tags,1,-1 do
+                                       local tag = stanza.tags[i];
+                                       if tag.name == "x" and tag.attr.xmlns == "http://jabber.org/protocol/muc#user" then
+                                               muc_child = tag;
+                                               break;
+                                       end
+                               end
+                       end
+               end
+       end
+       if muc_child then
+               for _, item in pairs(muc_child.tags) do
+                       if item.name == "item" then
+                               if from_occupant == to_occupant then
+                                       item.attr.jid = stanza.attr.to;
+                               else
+                                       item.attr.jid = from_occupant.jid;
+                               end
+                       end
+               end
+       end
+       self:route_stanza(stanza);
+       if muc_child then
+               for _, item in pairs(muc_child.tags) do
+                       if item.name == "item" then
+                               item.attr.jid = nil;
+                       end
+               end
+       end
+end
+
 local _M = {}; -- module "muc"
 
 function _M.new_room(jid)