Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

VOIP calls to encrypted rooms with unverified devices stuck on "Connecting Call..." #1581

Closed
repli-a opened this issue Oct 16, 2017 · 3 comments
Closed

VOIP calls to encrypted rooms with unverified devices stuck on "Connecting Call..." #1581

repli-a opened this issue Oct 16, 2017 · 3 comments
Labels
A-E2EE P1 T-Defect Something isn't working: bugs, crashes, hangs and other reported problems

Comments

@repli-a
Copy link

repli-a commented Oct 16, 2017 •
edited
Loading

Calls to E2E rooms with unverified devices will hang at "Connecting Call..."

Upon verifying any unverified devices the iOS end will be able to successfully connect the call.

This seems to only happen with iOS clients.
@manuroe
Copy link
Member

manuroe commented Oct 16, 2017

Related to element-hq/riot-android#1674

@dva-re
Copy link

dva-re commented Oct 16, 2017

Tried just now. All devices are confirmed. But the call hangs on the phrase "Connecting Call..." anyway.

Devices:

  1. Android, latest dev-build
  2. iOS 11.0.3, Riot - 0.5.6
    Works only if we are in the same Wifi network or in one VPN network.

P.S. Android <-> Android or Android <-> Desktop (web) works fine with any network configuration.

@ara4n
Copy link
Member

ara4n commented Oct 17, 2017

This sounds like the same as element-hq/element-web#2669.

@manuroe - this has been broken for almost a year; please can we get it fixed as part of the upcoming E2E work?

@ara4n ara4n added T-Defect Something isn't working: bugs, crashes, hangs and other reported problems A-E2EE P1 labels Oct 17, 2017
@manuroe manuroe mentioned this issue Oct 17, 2017
Merged
manuroe added a commit to matrix-org/matrix-ios-sdk that referenced this issue Oct 17, 2017
@manuroe
MXRoom: sendEventOfType: Copy the event content to send to keep it co…
f09e80b 
…nsistent in multi-thread conditions (like in e2e)

element-hq/element-ios#1581
In a e2e room, this fix fixes a race condition when answering a call. The race was between the ICE candidates gathering and the encryption of "m.call.candidates" event content.
@manuroe manuroe closed this as completed Oct 18, 2017
This was referenced Nov 3, 2017
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-E2EE P1 T-Defect Something isn't working: bugs, crashes, hangs and other reported problems
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@ara4n @manuroe @dva-re @repli-a