1624673 - Inconsistent serialization of rtpmap for telephone-event

Status: NEW

(Core :: WebRTC: Signaling, defect, P3)

Description

[Byron Campen [:bwc]]

https://jsfiddle.net/2fL598wx/1/

See console output. SDP from createOffer looks like "a=rtpmap:101 telephone-event/8000/1", but SDP from localDescription looks like "a=rtpmap:101 telephone-event/8000"

Spotted over in bug 1624006.

Comment 1

[Byron Campen [:bwc]]

I think this might have something to do with the fact that telephone-event isn't handled in this code:

https://searchfox.org/mozilla-central/rev/c80fa7258c935223fe319c5345b58eae85d4c6ae/media/webrtc/signaling/src/sdp/SipccSdpMediaSection.cpp#309-324

Comment 2

[Byron Campen [:bwc]]

Nico, might this be a source of inconsistency between the rust and sipcc parsers?

Comment 3

[Nico Grunbaum [:ng, @chew:mozilla.org]]

(In reply to Byron Campen [:bwc] from comment #2)

Nico, might this be a source of inconsistency between the rust and sipcc parsers?

Yes it could!