elixir-webrtc / ex_webrtc

An Elixir implementation of the W3C WebRTC API
Apache License 2.0
269 stars 8 forks source link

Add support for RTCP Receiver Reports #77

Closed LVala closed 4 months ago

LVala commented 4 months ago

This PR only adds RTPReceiver.ReportRecorder module, akin to PeerConnection.TWCCRecorder. The actual functionality of sending the reports, as well as RTPSender.ReportRecorder for SenderReports, will be implemented in subsequent PRs.

codecov[bot] commented 4 months ago

Codecov Report

Merging #77 (ee60139) into master (682c434) will increase coverage by 0.37%. The diff coverage is 98.11%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #77 +/- ## ========================================== + Coverage 88.43% 88.81% +0.37% ========================================== Files 29 30 +1 Lines 1306 1359 +53 ========================================== + Hits 1155 1207 +52 - Misses 151 152 +1 ``` | [Files](https://app.codecov.io/gh/elixir-webrtc/ex_webrtc/pull/77?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=elixir-webrtc) | Coverage Δ | | |---|---|---| | [lib/ex\_webrtc/rtp\_receiver/report\_recorder.ex](https://app.codecov.io/gh/elixir-webrtc/ex_webrtc/pull/77?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=elixir-webrtc#diff-bGliL2V4X3dlYnJ0Yy9ydHBfcmVjZWl2ZXIvcmVwb3J0X3JlY29yZGVyLmV4) | `98.11% <98.11%> (ø)` | | ------ [Continue to review full report in Codecov by Sentry](https://app.codecov.io/gh/elixir-webrtc/ex_webrtc/pull/77?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=elixir-webrtc). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=elixir-webrtc) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://app.codecov.io/gh/elixir-webrtc/ex_webrtc/pull/77?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=elixir-webrtc). Last update [682c434...ee60139](https://app.codecov.io/gh/elixir-webrtc/ex_webrtc/pull/77?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=elixir-webrtc). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=elixir-webrtc).