Skip to content

add secondary failure catch#613

Merged
stefpi merged 2 commits into
masterfrom
no-candidate-error
Jun 5, 2026
Merged

add secondary failure catch#613
stefpi merged 2 commits into
masterfrom
no-candidate-error

Conversation

@stefpi
Copy link
Copy Markdown
Contributor

@stefpi stefpi commented Jun 5, 2026

  • add timeout on no "connected" event, which implies that all ICE candidates found on browser side were rejected and no other ICE candidates were found.
  • create progress enums
  • fast exit option from gather step if srflx or `relay option is found
    • until trickle ice is implemented this significantly speeds up connection, especially on Chrome since Chrome implementation of 'icegatheringstatechange' only sends completed event once the connection has already been formed.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 5, 2026

Welcome to connect! Make sure to:

  • read the contributing guidelines
  • mark your PR as a draft until it's ready to review
  • post the preview on Discord; feedback from users will speedup the PR review

deployed preview: https://613.connect-d5y.pages.dev

@stefpi stefpi marked this pull request as ready for review June 5, 2026 18:40
@stefpi stefpi merged commit da60950 into master Jun 5, 2026
4 checks passed
@stefpi stefpi deleted the no-candidate-error branch June 5, 2026 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant