Revert STUN checking fix

kylecarbs · kylecarbs · commit 75e066387c99 · 2022-09-04T20:55:35.000Z
diff --git a/net/netcheck/netcheck.go b/net/netcheck/netcheck.go
@@ -880,18 +880,10 @@ func (c *Client) GetReport(ctx context.Context, dm *tailcfg.DERPMap) (_ *Report,
 	for _, probeSet := range plan {
 		setCtx, cancelSet := context.WithCancel(ctx)
 		go func(probeSet []probe) {
-			pwg := syncs.NewWaitGroupChan()
-			pwg.Add(len(probeSet))
-			for _, p := range probeSet {
-				go func(probe probe) {
-					rs.runProbe(setCtx, dm, probe, cancelSet)
-					pwg.Decr()
-				}(p)
-			}
-			select {
-			case <-setCtx.Done():
-			case <-pwg.DoneChan():
+			for _, probe := range probeSet {
+				go rs.runProbe(setCtx, dm, probe, cancelSet)
 			}
+			<-setCtx.Done()
 			wg.Decr()
 		}(probeSet)
 	}
