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.

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

If two people make a call at the same time, the state of the call will be broken #17418

Closed
smorimoto opened this issue May 20, 2021 · 10 comments
Closed
Labels
A-VoIP P1 T-Defect X-Needs-Info This issue is blocked awaiting information from the reporter

Comments

@smorimoto
Copy link

Description

As the title says.

Steps to reproduce

  1. User A makes a call
  2. User B makes a call
  3. User A ends own call
  4. User B ends own call
  5. The Element client continues to ring the call bell

Logs being sent

No.

Version information

  • Platform: Desktop

For the desktop app:

  • OS: Windows
  • Version:
    • Element version: 1.7.28
    • olm version: 3.2.1
@robintown
Copy link
Member

This sounds like #17393, which should be fixed in 1.7.29rc1

@smorimoto
Copy link
Author

You may be right. Let me try that tomorrow evening.

@robintown
Copy link
Member

robintown commented May 21, 2021

Wait I must be wrong, #17393 never got past staging…

@SimonBrandner
Copy link
Contributor

I think I might have seen this in the wild before 🤔 Probably even before my CallFeed rewrite

@SimonBrandner
Copy link
Contributor

@smorimoto, logs might actually help in this case. Could you please send them?

@t3chguy
Copy link
Member

t3chguy commented May 21, 2021

This sounds like #17393, which should be fixed in 1.7.29rc1

It doesn't look to have made it into any RC yet.

image

only available on /develop

matrix-org/matrix-js-sdk#1690 looks related and this is in staging - would be good if reporter could retry on https://develop.element.io

@t3chguy t3chguy added the X-Needs-Info This issue is blocked awaiting information from the reporter label May 21, 2021
@SimonBrandner
Copy link
Contributor

This sounds like #17393, which should be fixed in 1.7.29rc1

It doesn't look to have made it into any RC yet.

image

only available on /develop

matrix-org/matrix-js-sdk#1690 looks related and this is in staging

This was discovered in 1.7.28

@SimonBrandner
Copy link
Contributor

matrix-org/matrix-js-sdk#1690 looks related and this is in staging - would be good if reporter could retry on develop.element.io

Related but the problem was something a bit different there but yeah would be great to try on develop

@smorimoto
Copy link
Author

I don't have enough time to confirm it, so it would be great if someone could confirm that! If necessary.

@SimonBrandner
Copy link
Contributor

Thank you for opening an issue. Unfortunately, I wasn’t able to reproduce it. If you keep experiencing this defect, please update your description with steps/screenshots/video/more details and add a comment with @SimonBrandner to let me know that it’s ready to be reopened.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-VoIP P1 T-Defect X-Needs-Info This issue is blocked awaiting information from the reporter
Projects
None yet
Development

No branches or pull requests

4 participants