[Bug 21093] New: Approver: AddDispatchOperation assumes CDO. Channels is immutable

[bugzilla-daemon at freedesktop.org]

http://bugs.freedesktop.org/show_bug.cgi?id=21093

           Summary: Approver: AddDispatchOperation assumes CDO.Channels is
                    immutable
           Product: Telepathy
           Version: unspecified
          Platform: Other
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: medium
         Component: telepathy-spec
        AssignedTo: telepathy-bugs at lists.freedesktop.org
        ReportedBy: simon.mcvittie at collabora.co.uk


The Channels property on ChannelDispatchOperation objects is mutable (channels
can fall off). However, AddDispatchOperation documentation implies that it is
immutable.

We should separate the Channels into a separate argument, and document the
inherent race conditions. For reference, those races are:

* channels that close immediately after AddDispatchOperation go bad - the
approver should call a method on the channel (i.e. make a TpChannel) to notice
whether they're dead or not

* channels that close immediately after HandleChannels is called - the handler
should do the same


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