Fix NACKs to more gracefully handle RTP sequence # rollover

Glimesh / janus-ftl-plugin

A plugin for the Janus WebRTC gateway to enable relaying of audio/video streams utilizing Mixer's FTL (Faster-Than-Light) protocol.

GNU Affero General Public License v3.0

44 stars 11 forks source link

Closed haydenmc closed 3 years ago

danstiner commented 3 years ago

My rough plan is to:

Merge #122 so we can haveextended sequence numbers that do not rollover
Extract the existing NACK code into a new class and add tests, maybe named PacketBuffer, #129
Enable NACKs behind a feature flag so it can be tested for as long as needed before fully rolling it out

danstiner commented 3 years ago

Will follow up with tests later as part of improving metrics.