end2end tests 12 & 14 are disabled at the moment, fix them #132
Assignees
Labels
bug
Something isn't working
Comments
weissi
pushed a commit
to weissi/swift-nio
that referenced
this issue
Jun 13, 2020
Motivation: The HTTP2StreamMultiplexer currently passes frames from child streams through to the rest of the pipeline, and does the same with flushes. Because HTTP2 is multiplexed, it is highly likely that in any channelRead cycle multiple child channels will make sufficient progress to want to flush their results. This means we can end up with multiple, redundant, flush calls. It would be better in general to try to reduce the amount of flushing necessary to no more than once per channelRead cycle. Modifications: - Keep track of the channelRead cycle and where we are in it. - Avoid flushing when we can expect a channelReadComplete. - Flush on channelReadComplete if necessary. Result: Fewer flushes, better performance under high read load.
weissi
pushed a commit
to weissi/swift-nio
that referenced
this issue
Feb 3, 2024
* SOCKS handshake handler implementation * Soundness * Remove placeholder text * Sad path tests * Soundness * Docs * Fix workflow tests * Make handler removable * Protect methods * Prevent test crashes * Cleanup public types * Add test that writing after auth fails * Add force handler removal tests * Remove client and server state from public api * Explicitly handle states * Remove promises * Fix test * Add data to authentication complete * Refactor to add authentication complete flag
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
see title, for @weissi to do as he promised that a while ago and then forgot :|
The text was updated successfully, but these errors were encountered: