w3c / webrtc-encoded-transform

WebRTC Encoded Transform
https://w3c.github.io/webrtc-encoded-transform/
Other
122 stars 26 forks source link

SDP mangling: control encoding or control packetization? #199

Closed alvestrand closed 9 months ago

alvestrand commented 10 months ago

(Unusual usage of bug tracker: asking for input on a feature of an outstanding PR) While pondering #186 , I wonder if I should redesign.

The purpose of having a packetizer field in the codec description is to control the packetizer/depacketizer once the codec has been negotiated. But putting it in the SDP description kind of mixes the SDP negotiation (outside of transport) with the transport control.

This came to a head when considering how to set the outgoing codec, given that we have a new API for setting it using SetParameters.

New suggestion:

It seems cleaner.

@jan-ivar @youennf @orphis @henbos - WDYT?

alvestrand commented 9 months ago

Switched to setting packetizer in today's version of #186