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

Multiple calls can be placed between participants #573

Closed
dbkr opened this issue Sep 8, 2022 · 2 comments
Closed

Multiple calls can be placed between participants #573

dbkr opened this issue Sep 8, 2022 · 2 comments
Labels
T-Defect Something isn't working: bugs, crashes, hangs, vulnerabilities, or other reported problems

Comments

@dbkr
Copy link
Member

dbkr commented Sep 8, 2022

Steps to reproduce

We've seen an instance of a client placing two calls to a peer in fairly quick succession. It's not 100% clear how, but presumably the member state event processor function must have got run again and we're not smart enough to realise we're already placing a call. The fix is probably to check for an existing call before placing another.

Outcome

What did you expect?

What happened instead?

Operating system

No response

Browser information

No response

URL for webapp

No response

Will you send logs?

Yes

@dbkr dbkr added the T-Defect Something isn't working: bugs, crashes, hangs, vulnerabilities, or other reported problems label Sep 8, 2022
@dbkr
Copy link
Member Author

dbkr commented Sep 8, 2022

Connectivity issues may be a factor in triggering this.

@dbkr
Copy link
Member Author

dbkr commented Sep 14, 2022

Should be fixed by matrix-org/matrix-js-sdk#2662

@dbkr dbkr closed this as completed Sep 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Defect Something isn't working: bugs, crashes, hangs, vulnerabilities, or other reported problems
Projects
None yet
Development

No branches or pull requests

1 participant