レビュー - コミュニティにDiscord風のビデオ・ボイスルームや通話を追加 🎙

Falco · 2026 年 7 月 30 日午後 7:09

詳細なレポートをありがとうございます。それらに関するPRをマージしました：

github.com/discourse/resenha

FIX: Presence race conditions, admin room_type handling, and LiveKit log noise (#72)

main ← fix-presence-and-room-type-bugs

merged 06:05PM - 30 Jul 26 UTC

xfalcox

+282 -3

This addresses five user-reported bugs around presence tracking, room types, and… LiveKit reconciliation. One commit per fix. ## 1. Racing heartbeats resurrected departed users (ghost presence) A heartbeat already in flight when a `leave` or `kick` was processed landed second and unconditionally re-added the departed user's presence. Because the TTL reap never broadcasts, the ghost stayed visible to other clients until the next republish sweep — up to a minute. Leave and kick now write a short-lived Redis tombstone (15s — one heartbeat interval plus latency) that `heartbeat` checks before re-adding presence. `join` and the reconnect ladder's `livekit_token` endpoint clear it, since both are explicit statements that the user is (re-)entering the call. ## 2. `room_type` could not be set through the admin UI, and unknown values silently became "open" The admin controller never permitted `room_type` and the admin serializer never returned it, so the admin form always displayed (and submitted) "open" — stage assignments made there were silently dropped. Both halves land together deliberately: permitting the param while the form still prefills "open" from the missing serializer field would flip existing stage rooms back to open on any admin edit. The user-facing controller also mapped unknown `room_type` values to open as a silent fallback. Both controllers now share a strict model-level lookup that rejects unknown names with a 400 — the column is an integer, so an unmapped name would otherwise cast to `0` (open) and corrupt the room. ## 3. A superseded LiveKit session's late departure webhook dropped a fresh session Departure webhooks were matched by user identity alone. On a quick disconnect/rejoin, the old session's `participant_left` routinely arrives after the new session is connected, and the `gone_at` heuristic can't tell them apart (the rejoin's presence timestamp predates the old session's disconnect), so the new session's presence was expired and the user vanished from the roster until their next heartbeat. `participant_joined` events now record the session SID per user, and a departure whose SID differs from the recorded one is skipped as superseded. Departures without SID context on either side still expire presence, keeping the webhook's reconcile-only backstop role intact. Webhook delivery is unordered, so this narrows the race window rather than sealing it — the next heartbeat still recovers the residual case. ## 4. The last leave's `DeleteRoom` race logged as a failure The SFU tears an emptied room down on its own, so the `DeleteRoom` issued by the last participant's leave routinely finds the room already gone and 404s. That's the desired end state, not a fault, yet it landed in Logster on every last-leave (and again from the emptied-room sweep). `DeleteRoom` now treats 404 as success at debug level; every other call and status keeps warning.

トピック		返信	表示
Voice: Discord-style voice and video rooms, now bundled with Discourse :studio_microphone: Announcements voice	46	1191	2026 年 9 月 10 日
A plugin for chat to connect to a video conferencing tool Feature chat , completed	2	659	2026 年 9 月 3 日
Introducing Discourse Chat (BETA) Announcements chat , new-feature	142	28250	2022 年 8 月 3 日
Discourse Chat Plugin official , chat , included-in-core	43	34460	2026 年 7 月 29 日
3.3.0.beta2: New admin sidebar, Redesigned bookmark menu, Chat from user profile, and more Announcements release-notes	1	2075	2024 年 5 月 15 日

レビュー - コミュニティにDiscord風のビデオ・ボイスルームや通話を追加 🎙

関連トピック