Bug 29656

Summary:	Call: Add support for RTP General header extensions signalling
Product:	Telepathy	Reporter:	Olivier Crête <olivier.crete>
Component:	tp-spec	Assignee:	Telepathy bugs list <telepathy-bugs>
Status:	RESOLVED FIXED	QA Contact:	Telepathy bugs list <telepathy-bugs>
Severity:	normal
Priority:	medium	CC:	david.laban
Version:	unspecified
Hardware:	Other
OS:	All
Whiteboard:	Call
i915 platform:		i915 features:

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 5 Olivier Crête 2011-02-10 03:59:03 UTC

Drafted:

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

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.