[Bug 33460] IM and MUC channels should use wocky_porter_send_async(), and emit failing delivery reports

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=33460

--- Comment #4 from Simon McVittie <simon.mcvittie at collabora.co.uk> 2011-04-29 05:39:19 PDT ---
(In reply to comment #2)
> (Approach 1) [...] and not bother with delivery reports at all.
> 
> (Approach 2) [...] emit a delivery report.
> 
> I think Approach 1 is the correct one for this situation: conceptually, the
> message was never sent, because we got disconnected before we could send it

13:36 < wjt> smcv: the advantage of the “return and emit MessageSent 
             immediately; emit a delivery report later” approach is that an 
             out-of-process logger can see the message that's failed to be sent
13:37 < wjt> but it's so much simpler to take the other approach that i think 
             you've convinced me (or i've convinced myself)
13:37 < smcv> oh yes, I knew there was some reason someone would advocate that
13:37 < smcv> but I couldn't remember what it was

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.