lynckia / licode

Open Source Communication Provider based on WebRTC and Cloud technologies
http://lynckia.com/licode
MIT License
3.1k stars 1.01k forks source link

FEC does not work #1639

Open vpoddubchak opened 4 years ago

vpoddubchak commented 4 years ago

Does licode support FEC ?

I see a lot of source code with handling of FEC, but looks like it does not work. I can see that answer from licode does not contain ulpfec/90000 and red/90000. I believe, licode removes it from Offer.

I tried to search in source code where it ignores FEC in Offer, and only places I found are:

and I'm not sure that this is correct places to modify...

Also I tried to modify rtp_media_config_default.js and set payload types to ulpfec - 123, red - 124 (I see those values in offer from Chrome) but it did not help.

Also, in another (closed) thread, I saw that FEC was disabled in purpose. If so, then why ? and is it possible to use it again ?

ihorbenko commented 3 years ago

Any update here?