[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 · dcc6a46 · dcc6a46
1 parent 0eb6625
commit dcc6a46
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
@@ -433,7 +433,16 @@ defmodule Jellyfish.Room do
         {peer_id, peer} ->
           :ok = Engine.remove_endpoint(state.engine_pid, peer_id)
           Event.broadcast_server_notification({:peer_disconnected, state.id, peer_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()