edgehog-device-manager / edgehog / 23585184560

76%

push

github

fix: Defer WebSocket creation to avoid close-before-connect warning

React StrictMode double-invokes effects in
development: `mount → unmount → remount`.
When the WebSocket was created synchronously inside the Observable
factory, the immediate unmount called ws.close() on a socket still
in CONNECTING state, producing an unavoidable browser-level warning
("WebSocket is closed before the connection is established").

Defer socket creation with setTimeout(0) so the cleanup function
can call clearTimeout() before any socket is ever opened, completely
eliminating the spurious connection attempt.

Signed-off-by: Omar <omar.brbutovic@secomind.com>

627 of 4402 branches covered (14.24%)

Branch coverage included in aggregate %.

0 of 28 new or added lines in 1 file covered. (0.0%)

1168 of 5937 relevant lines covered (19.67%)

1.89 hits per line