matrix-org / waterfall

A cascading stream forwarding unit for scalable, distributed voice and video conferencing over Matrix
Apache License 2.0
97 stars 5 forks source link

Revisit the RTP timestamp and seqNum rewriting logic #103

Closed daniel-abramov closed 1 year ago

daniel-abramov commented 1 year ago

There are 3 things that we don't take into account:

Also, currently, we don't properly expand the timestamp in case of a rollover: the timestamp gets properly extended, but once the sequence number reaches the sequence number with which the simulcast began, we don't properly expand the timestamp which creates some issues.