fix: Run status callbacks async to solve tailnet race

kylecarbs · kylecarbs · commit 53f8adb70456 · 2022-09-05T15:23:17.000Z
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)
+		}()
 	})
 }
 
