Skip to content

add transport-wide-cc feedback packet#24

Merged
ystreet merged 1 commit intoystreet:mainfrom
kbalt:transport-wide-cc
Dec 1, 2025
Merged

add transport-wide-cc feedback packet#24
ystreet merged 1 commit intoystreet:mainfrom
kbalt:transport-wide-cc

Conversation

@kbalt
Copy link
Contributor

@kbalt kbalt commented Nov 22, 2025

Adds transport-cc feedback packet parser & builder.

I tested it against itself and Wireshark also deems packets produced by this as valid.

Tried using it in a WebRTC connection with a browser but they don't seem to generate transport-cc feedback, so I couldn't verify that the parser works with that.

Let me know if you're interested in this and any if you have any feedback.

@kbalt kbalt force-pushed the transport-wide-cc branch 3 times, most recently from a77b2c7 to cd409e4 Compare November 22, 2025 16:45
@ystreet
Copy link
Owner

ystreet commented Nov 24, 2025

Very interested in this! A quick look seems to have all the right things but I'll have a closer look at this later this week.

@kbalt kbalt force-pushed the transport-wide-cc branch 2 times, most recently from 1c51fb8 to a23c4f3 Compare November 27, 2025 15:28
@ystreet
Copy link
Owner

ystreet commented Nov 27, 2025

The MSRV check needs an update of the Cargo.lock.msrv for the new rand dev-dependency.

@kbalt kbalt force-pushed the transport-wide-cc branch 2 times, most recently from 48e8451 to f27fc87 Compare November 29, 2025 00:47
@kbalt kbalt force-pushed the transport-wide-cc branch from f27fc87 to 545ef6f Compare November 29, 2025 00:48
@kbalt kbalt requested a review from ystreet November 29, 2025 01:05
@ystreet ystreet merged commit b751328 into ystreet:main Dec 1, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants