ibc / libsdptransform

Session Description Protocol C++ parser/writer based on the sdp-transform JavaScript library
MIT License
134 stars 56 forks source link

Media number of ports #10

Closed SebastienBruckert closed 5 years ago

SebastienBruckert commented 5 years ago

Hello, As described in the RFC here https://tools.ietf.org/html/rfc4566#section-5.14, there may be a number to describe how many ports the RTP/RTCP will use, like this: m=video 49170/2 RTP/AVP 31

Since in the json, the port is a number, it would loose everything after the /. Is there any way to add that ? Thank you !

ibc commented 5 years ago

Wow, I didn't know that. BTW, is that really used anywhere today? I can add a numPorts field, but I've never seen that usage in any SDP.

ibc commented 5 years ago

Done in 1.2.2.

SebastienBruckert commented 5 years ago

Perfect thank you for your reactivity ! I'm jut a SDP beginner, and I have never seen this neither ; but I'm looking at a lot of RFC in details. I 'll let you know if I found something else.