Skip to content

FCE-3295: Fix media event queue order catch#530

Merged
MiloszFilimowski merged 1 commit into
mainfrom
fix-queue
May 6, 2026
Merged

FCE-3295: Fix media event queue order catch#530
MiloszFilimowski merged 1 commit into
mainfrom
fix-queue

Conversation

@MiloszFilimowski

Copy link
Copy Markdown
Collaborator

Description

  • webRTCEndpoint.ts: chain .catch() before .then() on mediaEventQueue so a rejected prior event doesn't propagate the same rejection to every subsequent receiveMediaEvent caller.

Motivation and Context

Previous order assigned the caught queue back to mediaEventQueue but returned the uncaught next, so one failed handler would reject every queued event after it. Catching first keeps the queue alive and isolates failures to their own caller.

Documentation impact

  • Documentation update required
  • Documentation updated in another PR
  • No documentation update required

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to
    not work as expected)

@linear

linear Bot commented May 6, 2026

Copy link
Copy Markdown

@MiloszFilimowski MiloszFilimowski requested a review from Copilot May 6, 2026 16:40
@MiloszFilimowski MiloszFilimowski merged commit 4f91d86 into main May 6, 2026
6 checks passed
@MiloszFilimowski MiloszFilimowski deleted the fix-queue branch May 6, 2026 16:46
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