Re: [prosody-dev] [PATCH][0.10] muc: Allow clients to change multiple affiliations or roles at once (#345)

2017-04-17 Thread Matthew Wild
On 17 April 2017 at 19:34, Kim Alvefur wrote: > On Mon, Apr 17, 2017 at 02:32:22PM +0100, Matthew Wild wrote: >> - Treat the request as an atomic change: succeed for all, or fail >> all (i.e. if one of the JIDs is malformed, don't allow any of the >> other changes in the request

Re: [prosody-dev] [PATCH][0.10] muc: Allow clients to change multiple affiliations or roles at once (#345)

2017-04-17 Thread Kim Alvefur
On Mon, Apr 17, 2017 at 02:32:22PM +0100, Matthew Wild wrote: > - Treat the request as an atomic change: succeed for all, or fail > all (i.e. if one of the JIDs is malformed, don't allow any of the > other changes in the request to take place). This would require work > to either pre-verify the

Re: [prosody-dev] [PATCH][0.10] muc: Allow clients to change multiple affiliations or roles at once (#345)

2017-04-17 Thread Lennart Sauerbeck
On 17.04.2017 15:32, Matthew Wild wrote: > On 17 April 2017 at 13:43, Lennart Sauerbeck > wrote: >> - Regarding the error handling in lines 49, 55 and in-tree 801 (the last >> one is not in the patch because I did no changes there): Is it okay to >> just send multiple