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

Fix flakey SSH connection #19439

Merged
merged 3 commits into from
Oct 18, 2024
Merged

Fix flakey SSH connection #19439

merged 3 commits into from
Oct 18, 2024

Conversation

mikayla-maki
Copy link
Contributor

Fixes a bug due to the select! macro tossing futures that had partially read messages, causing us to desync our message reading with the input stream.

Release Notes:

  • N/A

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Oct 18, 2024
co-authored-by: mikayla <[email protected]>
@mikayla-maki mikayla-maki changed the title Fix select stuff Fix flakey SSH connection Oct 18, 2024
@mikayla-maki mikayla-maki merged commit 8a91272 into main Oct 18, 2024
11 checks passed
@mikayla-maki mikayla-maki deleted the fix-select-stuff branch October 18, 2024 22:41
@mrnugget
Copy link
Member

whoa!

noaccOS pushed a commit to noaccOS/zed that referenced this pull request Oct 19, 2024
Fixes a bug due to the `select!` macro tossing futures that had
partially read messages, causing us to desync our message reading with
the input stream.

Release Notes:

- N/A

---------

Co-authored-by: Conrad Irwin <[email protected]>
Co-authored-by: conrad <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants