Bug 28690 - Add the ability to do an ICE restart
Summary: Add the ability to do an ICE restart
Status: RESOLVED FIXED
Alias: None
Product: Telepathy
Classification: Unclassified
Component: tp-spec (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Telepathy bugs list
QA Contact: Telepathy bugs list
URL: http://git.collabora.co.uk/?p=user/jo...
Whiteboard: Call
Keywords: patch
Depends on:
Blocks:
 
Reported: 2010-06-23 06:17 UTC by Sjoerd Simons
Modified: 2010-11-03 07:34 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments

Description Sjoerd Simons 2010-06-23 06:17:57 UTC
From #24936

> * Add an signal to Stream.I.Media for the CM to request an ICE restart, which
> means the CM will forget all local candidates/credentials and the streaming
> implementation is expected to create new ones (but streaming will continue to
> the currently selected candidate).

Who would make initiate a restart of the ice session ? as in who would take that decision and why ?
Comment 1 Jonny Lamb 2010-10-26 04:10:09 UTC
Mr Crêpe says:
> If the RemoteCredentialsSet signal is fired again, and the credentials
> have changed, then there has been an ICE restart.
> Also, in the same case, the CM should empty the remote candidate list

> If you the CM does an ICE restart, then it fires PleaseRestartICE
> and tp-fs should reply with SetCredentials...
Comment 2 Jonny Lamb 2010-10-26 09:20:08 UTC
See my branch.
Comment 3 Jonny Lamb 2010-10-26 09:20:59 UTC
See my branch.
Comment 4 Olivier Crête 2010-10-29 05:46:13 UTC
You should probably reference http://tools.ietf.org/html/rfc5245#section-9.1.1.1 in the section about ICE restart.
Comment 5 Sjoerd Simons 2010-10-29 10:34:18 UTC
Branch looks good, although like Olivier says, referring to the ICE spec would be nice ;)
Comment 6 Sjoerd Simons 2010-11-03 07:34:48 UTC
merged


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.