Multi-user chat patches

Jeremy Lawler jeremylawler at gmail.com
Thu Sep 20 21:05:22 UTC 2007


I have 2 patches to multi-user chat.  Both are ridiculously small, but
also fix what I consider to be serious bugs.

The djabberd_patch_fix_leave_room fixes a problem where when someone
leaves a MUC room, we don't send the presence message back to the
person who left.  This results in clients who will not rejoin chats
because they believe they are still a member.

The djabberd_patch_fix_muc_disco fixes the problem that we don't
properly advertise a chat server to a disco#items query.  My patch
solves the problem, the reason I don't like my fix is that my patch
uses set_config_childservice.  The reason I used this is because there
was no other way (that I'm aware of) to modify the "disco_kids"
element of the vhost.  I didn't want to directly modify the contents
of the object, and I didn't want to add functions to VHost.pm.
Basically, I'm looking for feedback.



Also, I am currently modifying DJabberd to use in a call center.  The
major changes to functionality are:
     LDAP authentication (which has been implemented elsewhere)
     Dynamic generation of the roster using the contents of LDAP
     Chat logging (and assorted MUC fixes/features)
     Authorization style patches that will (among other things) allow
users to communicate with their respective bosses without allowing
them to communicate with each other.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: djabberd_patch_fix_leave_room
Type: application/octet-stream
Size: 483 bytes
Desc: not available
Url : http://lists.danga.com/pipermail/djabberd/attachments/20070920/a2ca461c/djabberd_patch_fix_leave_room.obj
-------------- next part --------------
A non-text attachment was scrubbed...
Name: djabberd_patch_fix_muc_disco
Type: application/octet-stream
Size: 471 bytes
Desc: not available
Url : http://lists.danga.com/pipermail/djabberd/attachments/20070920/a2ca461c/djabberd_patch_fix_muc_disco.obj


More information about the Djabberd mailing list