Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

[Backport staging] Fix call splitbrains when switching between rooms #9701

Merged
merged 1 commit into from
Dec 2, 2022

Conversation

RiotRobot
Copy link
Contributor

@RiotRobot RiotRobot commented Dec 2, 2022

Backport 62a740d from #9692.


Here's what your changelog entry will look like:

🐛 Bug Fixes

  • [Backport staging] Fix call splitbrains when switching between rooms (#9701). Contributed by @RiotRobot.

* Fix call splitbrains when switching between rooms

Mounting CallView causes the user's call membership room state to be cleaned up. However, because the GroupCall object always thought the local device was disconnected from the call, it would remove the local device from room state when the clean method is called, causing a splitbrain. This uses GroupCall's new enteredViaAnotherSession field to fix that, and also simplify participant tracking.

* Remove clean tests that have been moved to matrix-js-sdk

(cherry picked from commit 62a740d)
@RiotRobot RiotRobot requested a review from a team as a code owner December 2, 2022 08:36
@RiotRobot RiotRobot requested review from t3chguy and alunturner and removed request for a team December 2, 2022 08:36
@RiotRobot RiotRobot added T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems X-Release-Blocker This affects the current release cycle and must be solved for a release to happen labels Dec 2, 2022
@github-actions github-actions bot added the Z-Community-PR Issue is solved by a community member's PR label Dec 2, 2022
@t3chguy t3chguy merged commit 07577a6 into staging Dec 2, 2022
@t3chguy t3chguy deleted the backport-9692-to-staging branch December 2, 2022 10:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems X-Release-Blocker This affects the current release cycle and must be solved for a release to happen Z-Community-PR Issue is solved by a community member's PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants