Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Flub/reset rtt #1

Closed
wants to merge 4 commits into from
Closed

Flub/reset rtt #1

wants to merge 4 commits into from

Conversation

flub
Copy link
Owner

@flub flub commented Apr 25, 2024

flub added 3 commits April 19, 2024 12:22
This is a super hacky way to be able to reset the right bits of the
congestion controller.
@dignifiedquire
Copy link

@flub given the actual diff is not that large, maybe we should open a Draft PR to quinn directly to get some feedback/see if there is a chance we can avoid running a fork for too long

@flub
Copy link
Owner Author

flub commented Apr 29, 2024

@dignifiedquire Yep, we should try, though in the current form I guess it's unlikely because I'm explicitly making internal things public.

There was too much state in the path that was being thrown away.
Possibly there is not enough being reset right now, especially the mtu
might matter.  But it at least works.
@flub flub closed this May 2, 2024
flub added a commit that referenced this pull request May 2, 2024
This is mostly copied from upstream, with a bit of changes inspired by
our own usual CI setup.
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