Bug 29656 - Call: Add support for RTP General header extensions signalling
Summary: Call: Add support for RTP General header extensions signalling
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:
Whiteboard: Call
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-18 15:32 UTC by Olivier Crête
Modified: 2011-07-19 12:06 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments

Description Olivier Crête 2010-08-18 15:32:26 UTC
RFC 5285 defines "A General Mechanism for RTP Header Extensions", this includes a mecanism to negotiate a ID to type mapping. This needs to be signalled down and back up (as they are subject to SDP Offer/Answer negotiation).

So we should probably add it to the CodecOffer..

For each possibility, the relevant fields are:
ID (a uint), direction (sendrecv, recvonly, sendonly, inactive), URI
Comment 1 Olivier Crête 2010-08-18 15:33:24 UTC
To support this, avpf, rtcp xr, maybe we should have a asv in the CodecOffer or something.
Comment 2 Olivier Crête 2010-08-18 15:33:50 UTC
I forgot to mention, these are not per-codec, they are per Content.
Comment 3 Olivier Crête 2010-08-24 12:49:47 UTC
There can also be extension attributes for each one, so maybe we should add a asv. Although nothing in the RFC says they will be using the a=b form.
Comment 4 Jonny Lamb 2010-10-25 09:45:26 UTC
An array, containing:

 u: ID
 u: send/recv/both/none
 s: URI
 s: misc
Comment 6 Olivier Crête 2011-02-11 11:20:31 UTC
Remove Method

Port to MediaDescription

Remove the Local.. it is the answer
the Description always has the remote
Comment 7 Olivier Crête 2011-02-14 06:37:12 UTC
Drafted at:

http://git.collabora.co.uk/?p=user/tester/telepathy-spec.git;a=shortlog;h=refs/heads/rtp-hdrext-mediadesc

As for rtcp-fb and codecs, I think the Remote in the property should go
Comment 8 David Laban 2011-02-14 10:06:47 UTC
Again, I think s/Remote/Supported/

Also, I think you mean "list" below:
  68         A lost of remote header extensions which have been set

Otherwise, ++
Comment 9 David Laban 2011-03-02 10:55:45 UTC
I went for s/Remote// (a la rtcp-fb-mediadesc) instead, and pushed to:

http://git.collabora.co.uk/?p=user/alsuren/telepathy-spec.git;a=commitdiff;h=refs/heads/rtp-hdrext-mediadesc-ftfy
Comment 10 Olivier Crête 2011-03-03 10:51:42 UTC
++
Comment 11 David Laban 2011-03-07 10:20:41 UTC
Abusing Assigned to mean "merged in alsuren/call and available at http://people.freedesktop.org/~alsuren/telepathy-spec-call/spec/".

There was a trivial patch required compared to what was reviewed (http://git.collabora.co.uk/?p=user/alsuren/telepathy-spec.git;a=commitdiff;h=0fe25dbc2175557a4ade4db78154a186ffd4452f) to actually make the thing compile. I took the liberty of considering it insta-reviewed.
Comment 12 David Laban 2011-03-07 10:37:53 UTC
*cough*
Comment 13 David Laban 2011-07-19 12:06:40 UTC
Marged to master.


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.