Skip to content

Fix padding bit in forwarded packet. - #4690

Merged
boks1971 merged 1 commit into
masterfrom
raja_padding
Jul 19, 2026
Merged

Fix padding bit in forwarded packet.#4690
boks1971 merged 1 commit into
masterfrom
raja_padding

Conversation

@boks1971

Copy link
Copy Markdown
Contributor

Addresses #4689

We were probably missing a couple of bits with this

  1. Not affected for regular traffic like from browsers as it does not add padding, but special clients were affected.
  2. Probe packets were probably using wrong last byte as pion/rtp would have overwrriten with 0 because the header.PaddingSize for the newer versions were not set. That could have affected bandwidth estimation catch up.

Addresses #4689

We were probably missing a couple of bits with this
1. Not affected for regular traffic like from browsers as it does not
   add padding, but special clients were affected.
2. Probe packets were probably using wrong last byte as pion/rtp would
   have overwrriten with 0 because the header.PaddingSize for the newer
   versions were not set. That could have affected bandwidth estimation
   catch up.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@boks1971
boks1971 merged commit 366cadc into master Jul 19, 2026
10 checks passed
@boks1971
boks1971 deleted the raja_padding branch July 19, 2026 15:23
eleboucher pushed a commit to eleboucher/homelab that referenced this pull request Jul 31, 2026
…4 ➔ v1.13.5) (#1778)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [docker.io/livekit/livekit-server](https://github.com/livekit/livekit) | patch | `v1.13.4` → `v1.13.5` |

---

> ⚠️ **Warning**
>
> Some dependencies could not be looked up. Check the [Dependency Dashboard](issues/567) for more information.

---

### Release Notes

<details>
<summary>livekit/livekit (docker.io/livekit/livekit-server)</summary>

### [`v1.13.5`](https://github.com/livekit/livekit/releases/tag/v1.13.5)

[Compare Source](livekit/livekit@v1.13.4...v1.13.5)

##### Added

- add mock for testing region pins with API ([#&#8203;4691](livekit/livekit#4691))
- Add country to participant closing log ([#&#8203;4693](livekit/livekit#4693))
- Tests for down stream packet push. ([#&#8203;4692](livekit/livekit#4692))
- Add status code for twirp request latency prometheus metric ([#&#8203;4621](livekit/livekit#4621))
- Support more h264 profiles ([#&#8203;4708](livekit/livekit#4708))
- log high stream start latency. ([#&#8203;4714](livekit/livekit#4714))

##### Changed

- Use simulcast constructor for VP9 if simulcasted. ([#&#8203;4696](livekit/livekit#4696))
- send resolved ringing timeout ([#&#8203;4697](livekit/livekit#4697))
- Update go deps ([#&#8203;4645](livekit/livekit#4645))
- protocol deps for logging webhook status ([#&#8203;4699](livekit/livekit#4699))
- protocol update with webhook status logging ([#&#8203;4700](livekit/livekit#4700))
- Record subscribe stream start time in prometheus. ([#&#8203;4704](livekit/livekit#4704))
- A bit better counting for track publish. ([#&#8203;4707](livekit/livekit#4707))

##### Fixed

- Fix padding bit in forwarded packet. ([#&#8203;4690](livekit/livekit#4690))
- Spelling fixes ([#&#8203;4698](livekit/livekit#4698))
- Do not call telemetry listener under pending track lock. ([#&#8203;4706](livekit/livekit#4706))
- Fix getRefLayerRTPTimestamp off-by-one that can panic on max layer index ([#&#8203;4712](livekit/livekit#4712))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about these updates again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My4xMDEuMSIsInVwZGF0ZWRJblZlciI6IjQzLjEwMS4xIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJyZW5vdmF0ZS9jb250YWluZXIiLCJ0eXBlL3BhdGNoIl19-->

Reviewed-on: https://git.erwanleboucher.dev/eleboucher/homelab/pulls/1778
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants