internal/http3: wait until clientConn is unregistered when closing it The longtest in std is currently failing due to lingering goroutines at the end of tests. This seems to be occurring due to CloseIdleConnections being called before the clientConn had the chance to be fully unregistered from the transport. This results in CloseIdleConnections being unable to close the QUIC endpoint (thinking that there are still active connections). Make sure that clientConn.Close only returns after clientConn is unregistered from the transport to fix this. Unfortunately, this race condition seems to occur only when testing in std using fakenet. Therefore, no tests in this CL for now. For golang/go#78701 For golang/go#70914 Change-Id: I35c66b3e843be5f53ceef602d0ac456f6a6a6964 Reviewed-on: https://go-review.googlesource.com/c/net/+/820540 Reviewed-by: Nicholas Husin <husin@google.com> LUCI-TryBot-Result: golang-scoped@luci-project-accounts.iam.gserviceaccount.com <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Damien Neil <dneil@google.com>
This repository holds supplementary Go networking packages.
This repository uses Gerrit for code changes. To learn how to submit changes to this repository, see https://go.dev/doc/contribute.
The git repository is https://go.googlesource.com/net.
The main issue tracker for the net repository is located at https://go.dev/issues. Prefix your issue with “x/net:” in the subject line, so it is easy to find.