fix: Run status callbacks async to solve tailnet race (#3866)

kylecarbs · web-flow · commit 2fa77a9bbd12 · 2022-09-05T10:43:24.000-05:00
diff --git a/tailnet/conn.go b/tailnet/conn.go
@@ -276,11 +276,13 @@ func (c *Conn) SetNodeCallback(callback func(node *Node)) {
 		for _, addr := range s.LocalAddrs {
 			endpoints = append(endpoints, addr.Addr.String())
 		}
-		c.lastMutex.Lock()
-		c.lastEndpoints = endpoints
-		node := makeNode()
-		c.lastMutex.Unlock()
-		callback(node)
+		go func() {
+			c.lastMutex.Lock()
+			c.lastEndpoints = endpoints
+			node := makeNode()
+			c.lastMutex.Unlock()
+			callback(node)
+		}()
 	})
 }
 
