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

ICE failures on v0.2.7/0.2.8 #553

Closed
dbkr opened this issue Aug 25, 2022 · 1 comment
Closed

ICE failures on v0.2.7/0.2.8 #553

dbkr opened this issue Aug 25, 2022 · 1 comment
Assignees
Labels
O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Something isn't working: bugs, crashes, hangs, vulnerabilities, or other reported problems

Comments

@dbkr
Copy link
Member

dbkr commented Aug 25, 2022

Steps to reproduce

We're getting reports of fairly consistent "other people appearing as grey squares". Looking at the logs, this appears to be due to ICE failures.

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?

No

@dbkr dbkr added T-Defect Something isn't working: bugs, crashes, hangs, vulnerabilities, or other reported problems S-Major Severely degrades major functionality or product features, with no satisfactory workaround O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience labels Aug 25, 2022
dbkr added a commit to matrix-org/matrix-js-sdk that referenced this issue Aug 25, 2022
We were casting a POJO to an RTCIceCandidate for the dummy
end-of-candidates candidate, but #2473
started calling .toJSON() on these objects.

Store separately whether we've seen the end of candidates rather than
adding on a dummy candidate object.

A test for this will follow, but a) I want to get this fix out and
b) I'm currently rewriting the call test file to add typing.

Fixes element-hq/element-call#553
dbkr added a commit to matrix-org/matrix-js-sdk that referenced this issue Aug 26, 2022
* Fix ICE end-of-candidates messages

We were casting a POJO to an RTCIceCandidate for the dummy
end-of-candidates candidate, but #2473
started calling .toJSON() on these objects.

Store separately whether we've seen the end of candidates rather than
adding on a dummy candidate object.

A test for this will follow, but a) I want to get this fix out and
b) I'm currently rewriting the call test file to add typing.

Fixes element-hq/element-call#553

* Remove hacks for testing

* Switch if branches
@dbkr
Copy link
Member Author

dbkr commented Aug 26, 2022

Fixed by matrix-org/matrix-js-sdk#2622

@dbkr dbkr closed this as completed Aug 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Major Severely degrades major functionality or product features, with no satisfactory workaround 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