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

Add text on flow control deadlocks #5001

Open
martinthomson opened this issue Apr 26, 2023 · 3 comments
Open

Add text on flow control deadlocks #5001

martinthomson opened this issue Apr 26, 2023 · 3 comments
Labels
-bis An issue that is best addressed in a future revision of a published document -transport editorial An issue that does not affect the design of the protocol; does not require consensus.

Comments

@martinthomson
Copy link
Member

For a new revision of transport, it would be good to have general guidance text about flow control deadlocks. The best reference we have today is the somewhat specific text in Section 2.1.3 of RFC 9204 and that isn't very satisfactory. This has come up a few times in discussions about APIs for WebTransport, which replicates a lot of the QUIC stream mechanics.

@martinthomson martinthomson added editorial An issue that does not affect the design of the protocol; does not require consensus. -transport -bis An issue that is best addressed in a future revision of a published document labels Apr 26, 2023
@LPardue
Copy link
Member

LPardue commented Apr 26, 2023

WebTransport seems like the prime target audience for the applicability draft. Section 4.4 is titled "Flow Control Deadlocks" and is ~500 words.

So a few of questions:

  1. Is the text in RFC 9308 sufficiently descriptive? (i.e., does it address the OP issue)
  2. We could consider porting the text or referencing it in rfc900-bis work. However, if its current content and location is fine, can we close this?
  3. There may be some interplay between streams and datagrams (not flow controlled, but competition for congestion control, which can impact flow control). Would that be in scope of this issue, or something to track separate?

@martinthomson
Copy link
Member Author

Oh, I forgot about that. I think that I wrote it, even. A pointer might be all that is needed then.

@LPardue
Copy link
Member

LPardue commented Apr 26, 2023

Here's some archaeology for you, your original PR 5 years ago - #1927

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-bis An issue that is best addressed in a future revision of a published document -transport editorial An issue that does not affect the design of the protocol; does not require consensus.
Projects
None yet
Development

No branches or pull requests

2 participants