Skip to content

[UE5.6] Fix: SFU crashing due to _sctpStreamIds being null (#706)#724

Merged
Belchy06 merged 2 commits intoEpicGamesExt:UE5.6from
Belchy06:UE5.6
Nov 17, 2025
Merged

[UE5.6] Fix: SFU crashing due to _sctpStreamIds being null (#706)#724
Belchy06 merged 2 commits intoEpicGamesExt:UE5.6from
Belchy06:UE5.6

Conversation

@Belchy06
Copy link
Copy Markdown
Collaborator

Relevant components:

  • Signalling server
  • Common library
  • Frontend library
  • Frontend UI library
  • Matchmaker
  • Platform scripts
  • SFU

Problem statement:

The SFU is currently crashing due to the transport _sctpStreamIds member being null. MediaSoup no longer exposes this member. However, our code that required it is also no longer needed as the getNextSctpStreamId() method provides the same functionality.

Solution

Remove usages of _sctpStreamIds

Test Plan and Compatibility

Tested SFU and multi-peer with both Pixel Streaming and Pixel Streaming 2

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Nov 17, 2025

🦋 Changeset detected

Latest commit: 437b2b4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@epicgames-ps/pixelstreaming-sfu Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Belchy06 Belchy06 added auto-backport Used to specify we want a PR to auto backport to a branch, must be paired with auto-backport-to-UEX. auto-backport-to-UE5.5 labels Nov 17, 2025
@Belchy06 Belchy06 self-assigned this Nov 17, 2025
@Belchy06 Belchy06 merged commit 17861ba into EpicGamesExt:UE5.6 Nov 17, 2025
3 checks passed
github-actions bot pushed a commit that referenced this pull request Nov 17, 2025
* Fix: SFU crashing due to _sctpStreamIds being null. Code using this member isn't needed anymore

* Add changeset

(cherry picked from commit 17861ba)
@github-actions
Copy link
Copy Markdown
Contributor

💚 All backports created successfully

Status Branch Result
UE5.5

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

mcottontensor added a commit that referenced this pull request Nov 17, 2025
[UE5.5] Fix: SFU crashing due to _sctpStreamIds being null (#724)
@Belchy06 Belchy06 changed the title Fix: SFU crashing due to _sctpStreamIds being null [UE5.6] Fix: SFU crashing due to _sctpStreamIds being null (#706) Nov 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Used to specify we want a PR to auto backport to a branch, must be paired with auto-backport-to-UEX. auto-backport-to-UE5.5

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants