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.

Sign up for GitHub

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

Jump to bottom

Add interface to query stream flow control #31

Closed
martinthomson opened this issue May 28, 2019 · 0 comments
Closed

Add interface to query stream flow control #31

martinthomson opened this issue May 28, 2019 · 0 comments
Assignees
@agrover

Comments

@martinthomson
Copy link
Member

In h3, it's possible that the qpack encoder stream could get blocked. When that happens, we will want to send literals rather than block. If we block on the encoder stream, we also have to block the request stream. In order to test this, we need a way to ask the stream how much flow control credit it has.

This should also check the connection-level flow control.
@martinthomson martinthomson mentioned this issue May 28, 2019
Closed
@agrover agrover self-assigned this May 28, 2019
agrover pushed a commit to agrover/neqo that referenced this issue May 29, 2019
transport: Add Connection::stream_credit_avail_send()
ac7d97f 
Gives a way for applications to know at least how much they can send
without hitting flow control. (They may still hit pacing and tx buffer
limits, of course.)

fixes mozilla#31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@agrover agrover

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@agrover @martinthomson