Stream recv flow-control bounded

nitely / nim-hyperx

Pure Nim http2 client and server 🖖

MIT License

24 stars 0 forks source link

Stream recv flow-control bounded #12

Closed nitely closed 3 months ago

nitely commented 3 months ago

fixes #6

[x] Enable flow-control (remove high limit)
[x] Move window update to recvBody
[x] Add window size defined value
[x] flow-control error on recv window size exceeded
[ ] Add functional concurrent tests that go over the window size

nitely commented 3 months ago

mmh for stream window updates, the stream can be close for whatever reason and it won't affect the rest. Main stream window updates seem tricky, as we must ensure the window update is sent if the stream is closed, or an error occurs and there is pending recv data.

We can keep the current behaviour for the main stream, but then consume space can grow up to number streams multiplied by window size. Instead of just window size.