[RTC-498] Fix tracks present after peer disconnect (#164)

* Refactor server_notification_test.exs * Refactor v2 * Remove tracks when peer disconnects * Remove comment * Revert unnecessary change
fishjam-dev · Mar 8, 2024 · 95e0bfa · 95e0bfa
1 parent 16996d3
commit 95e0bfa
Show file tree

Hide file tree

Showing 2 changed files with 223 additions and 137 deletions.
diff --git a/lib/jellyfish/room.ex b/lib/jellyfish/room.ex
@@ -435,7 +435,16 @@ defmodule Jellyfish.Room do
           :ok = Engine.remove_endpoint(state.engine_pid, peer_id)
           Event.broadcast_server_notification({:peer_disconnected, state.id, peer_id})
           :telemetry.execute([:jellyfish, :room], %{peer_disconnects: 1}, %{room_id: state.id})
-          peer = %{peer | status: :disconnected, socket_pid: nil}
+
+          peer.tracks
+          |> Map.values()
+          |> Enum.each(
+            &Event.broadcast_server_notification(
+              {:track_removed, state.id, {:peer_id, peer_id}, &1}
+            )
+          )
+
+          peer = %{peer | status: :disconnected, socket_pid: nil, tracks: %{}}
 
           put_in(state, [:peers, peer_id], peer)
           |> maybe_schedule_peerless_purge()