Fix channels being advertised twice to new clients (#9)

### Changelog Fixed an issue where channels were being advertised twice to new clients. ### Docs None ### Description This was happening for both `.preparing` and `.ready` states. --------- Co-authored-by: Richard Dancsi <[email protected]>
foxglove · Jul 2, 2024 · 6fcaf42 · 6fcaf42
1 parent 088b1c1
commit 6fcaf42
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/WebSocketDemo-Shared/FoxgloveServer.swift b/WebSocketDemo-Shared/FoxgloveServer.swift
@@ -134,20 +134,20 @@ class FoxgloveServer: ObservableObject {
       connection.stateUpdateHandler = { state in
         print("connection state \(state)")
 
-        if connection.state == .ready {
+        if state == .ready {
           self.sendInfo(connection)
         }
 
         let closed: Bool
-        switch connection.state {
+        switch state {
         case .cancelled, .failed:
           closed = true
         default:
           closed = false
         }
 
         Task { @MainActor in
-          if !self.channels.isEmpty {
+          if state == .ready, !self.channels.isEmpty {
             try self.sendJson([
               "op": "advertise",
               "channels": self.channels.values.map {