-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
race condition in TestProtoDowngrade #1144
Comments
|
(reopen if I'm missing something) |
Reopening because the PR in quic-go is not merged yet, and the update is not bubbled up yet. Also, there's an alternative fix in quic-go: quic-go/quic-go#3241. |
This was fixed when we update go-libp2p-quic-transport to v0.12.0. |
See related quic-go issue: quic-go/quic-go#3239.
I'm not sure if we're using the API here correctly. Calling
Read
andCancelRead
concurrently doesn't make a lot of sense.It seems easy to fix though, so maybe we can get away with it?
The text was updated successfully, but these errors were encountered: