Bug 26270 - sends session-accept before the Tp user has responded
Summary: sends session-accept before the Tp user has responded
Status: RESOLVED FIXED
Alias: None
Product: Telepathy
Classification: Unclassified
Component: fargo (show other bugs)
Version: git master
Hardware: Other All
: high normal
Assignee: Simon McVittie
QA Contact: David Laban
URL: http://git.collabora.co.uk/?p=user/sm...
Whiteboard: milestone4.7 est=7h
Keywords: patch
Depends on: 26413
Blocks:
  Show dependency treegraph
 
Reported: 2010-01-27 06:11 UTC by Simon McVittie
Modified: 2010-02-18 14:21 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments

Description Simon McVittie 2010-01-27 06:11:20 UTC
# FIXME: should Fargo really accept the call before the XMPP client
            # has accepted the call?
            # In this case, it's the only way to get the codecs out of
            # telepathy so we can send session-initiate, so yes.
            EventPattern('dbus-method-call', handled=False,
                interface=cs.CHANNEL_IFACE_GROUP, method='AddMembers',
                path=chan.object_path, args=[[conn.self_handle], '']),

14:00 < smcv> alsuren: er, is it really necessary to call AddMembers before 
              doing all the codec wibbling?
14:00 < smcv> alsuren: I would expect that that ought to work even while the 
              local user is still in local-pending state

14:07 < alsuren> smcv: I was going by the description on: 
file:///usr/share/gtk-doc/html/telepathy-spec/org.freedesktop.Telepathy.Channel.Type.StreamedMedia.html. I guess you could set the stream direction yourself 
                 before you accept the call
14:08 < alsuren> not tried it though
14:08 < smcv> mm
14:08 < smcv> I should try that
14:09 < smcv> it doesn't look very good for a user of this gateway if Fargo 
              claims the XMPP user has answered the call, before they actually 
              have
14:09 < smcv> the SIP user will potentially be going "hello? hello? you there?" 
              for several seconds
Comment 1 Simon McVittie 2010-01-27 09:44:38 UTC
7h? It's just a matter of re-ordering events, but they're quite subtle events and will need careful testing.
Comment 2 Simon McVittie 2010-02-18 05:26:49 UTC
smcv/acceptance has patches.
Comment 3 Simon McVittie 2010-02-18 10:32:31 UTC
Fixed in git
Comment 4 Simon McVittie 2010-02-18 14:07:36 UTC
The reverse of the originally-reported problem has been reinstated as a hackish way to get calls working again.
Comment 5 Simon McVittie 2010-02-18 14:21:08 UTC
... or, in fact, not, because Sjoerd is ninja.


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.