zarazaex69 82e8067384 fix(client,server): defer shutdown BEFORE bringUpLink to close MUC on early failures ...

When bringUpLink errored — a handshake timeout against a wedged transport,
for instance — Run/RunWithReady returned straight to the caller without
calling shutdown, so the carrier link that had already joined the MUC
was never closed. The result was a ghost participant lingering on
Jicofo/JVB until idle timeout, which the next test in the same room
inherited as stale endpoints in 'bridge open'.

The clue from logs was that failing seichannel runs produced one
'leave-muc handshake ok' instead of two: the server's normal
ctx-cancel path got there cleanly, but the client's bringUpLink
returned early and skipped its defer.

Both paths now register shutdown before the bringUpLink call. shutdown
is nil-safe and idempotent so it works whether or not bringUpLink
actually populated link/session fields. server's wg.Wait moves into
the same defer so wg goroutines spawned by partial setup also drain
before Run returns.

2026-05-16 01:38:52 +03:00

..

server_test.go

fix: golangci lint fix

2026-05-14 04:21:11 +03:00

server.go

fix(client,server): defer shutdown BEFORE bringUpLink to close MUC on early failures

2026-05-16 01:38:52 +03:00