When you try to join a room uninvited, and it's full, the request correctly fails with the Channel.Full error. However, if you're invited to a room, and you try to accept the invitation and the room's full, you get removed from the group interface for reason None and then the channel spontaneously closes. In TpChannel terms, the invalidation error is TP_ERROR_REMOVED_FROM_GROUP, TP_CHANNEL_GROUP_CHANGE_REASON_NONE, which is unhelpful. My patch in busy-kind-of-means-full exposes this as Group_Change_Reason_Busy, which seems like a reasonable approximation. A better fix might be: • Make Gabble include the Channel.Full error (and indeed other errors like Invite_Only) in the MembersChangedDetailed details; • Make TpChannel smarter about what it saves from the MembersChanged[Detailed] signal so that it can synthesise better error messages. But using a slightly more useful group change reason doesn't hurt anyway.
(In reply to comment #0) > My patch in busy-kind-of-means-full exposes this as Group_Change_Reason_Busy, > which seems like a reasonable approximation. That looks OK to merge. > A better fix might be: > > • Make Gabble include the Channel.Full error (and indeed other errors like > Invite_Only) in the MembersChangedDetailed details; > • Make TpChannel smarter about what it saves from the > MembersChanged[Detailed] > signal so that it can synthesise better error messages. This is part of Bug #23369, for which I have a rather bit-rotted branch.
(In reply to comment #1) > (In reply to comment #0) > > My patch in busy-kind-of-means-full exposes this as Group_Change_Reason_Busy, > > which seems like a reasonable approximation. > > That looks OK to merge. Merged to 0.8 as http://git.collabora.co.uk/?p=telepathy-gabble.git;a=commit;h=bde36dba and master as http://git.collabora.co.uk/?p=telepathy-gabble.git;a=commit;h=7ee0588206
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.