webrtc-rs / webrtc

A pure Rust implementation of WebRTC
https://webrtc.rs
Apache License 2.0
4.08k stars 363 forks source link

Fix packet padding when changing extension #474

Closed rogurotus closed 1 year ago

rogurotus commented 1 year ago

in #472 I missed the cases when the extension is removed or replaced

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 19.04% and project coverage change: -0.11% :warning:

Comparison is base (2c189d6) 61.67% compared to head (9930780) 61.56%. Report is 2 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #474 +/- ## ========================================== - Coverage 61.67% 61.56% -0.11% ========================================== Files 541 541 Lines 49927 49934 +7 Branches 12397 12410 +13 ========================================== - Hits 30790 30740 -50 - Misses 9926 9952 +26 - Partials 9211 9242 +31 ``` | [Files Changed](https://app.codecov.io/gh/webrtc-rs/webrtc/pull/474?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=webrtc-rs) | Coverage Δ | | |---|---|---| | [rtp/src/header.rs](https://app.codecov.io/gh/webrtc-rs/webrtc/pull/474?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=webrtc-rs#diff-cnRwL3NyYy9oZWFkZXIucnM=) | `60.69% <19.04%> (-3.49%)` | :arrow_down: | ... and [29 files with indirect coverage changes](https://app.codecov.io/gh/webrtc-rs/webrtc/pull/474/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=webrtc-rs)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.