Implement NACK+RTX suppport #45
Conversation
|
Hi Sergio, thank you for your PR. Could you please provide some background information for this PR in the description? The patch will be generated based on the title and description, and it will be sent via email to the FFmpeg community. Therefore, a detailed description is important for the maintainers to review this PR. |
Signed-off-by: Jack Lau <jacklau1222@qq.com>
|
RTP retransmission described in RFC4588 (RTX) is an effective packet loss recovery technique for real-time applications with relaxed delay bounds. This PR provides a minimal implementation for RTX and RTCP NACK (RFC3940) and its associated SDP signaling and negotiation. |
winlinvip
force-pushed
the
workflows
branch
3 times, most recently
from
3294e09 to
4aa17ba
Compare
decrypt the SRTCP pakcet, get the right pid and blp data add more comments Signed-off-by: Jack Lau <jacklau1222@qq.com>
Signed-off-by: Jack Lau <jacklau1222@qq.com>
Signed-off-by: Jack Lau <jacklau1222@qq.com>
JackLau1222
force-pushed
the
nack-rtx
branch
2 times, most recently
from
f655237 to
ec1b1bf
Compare
unique ssrc need to use unique srtp context, otherwise the server couldn't decrypted the rtx packet. move the "a=ssrc-group:FID" on top of SSRCs Signed-off-by: Jack Lau <jacklau1222@qq.com>
Signed-off-by: Jack Lau <jacklau1222@qq.com>
Signed-off-by: Jack Lau <jacklau1222@qq.com>
Signed-off-by: Jack Lau <jacklau1222@qq.com>
whip->buf size is 4096 rather than srtcp_len, srtcp data just use part of its memory, handle it need more logic. So allocate a temporary pkt to carry the SRTCP data Signed-off-by: Jack Lau <jacklau1222@qq.com>
need to be changed AV_LOG_DEBUG before official commit Signed-off-by: Jack Lau <jacklau1222@qq.com>
Good newsThis patch has been tested with Pion and Janus, and it works well. However, this patch need to depend on some other patch(not in FFmepg master for now) otherwise you might run into problems, so I put these patches here: In the coming days, I'll add pion and janus tests into workflow, improve this patch, then submit this patch to FFmpeg developers mail list |
RTP retransmission described in RFC4588 (RTX) is an effective packet loss recovery technique for real-time applications with relaxed delay bounds.
This PR provides a minimal implementation for RTX and RTCP NACK (RFC3940) and its associated SDP signaling and negotiation.