Skip to content

[UE5.6] fix(signalling): leave the previous streamer when an SFU subscribes again (#1156) - #1172

Merged
mcottontensor merged 2 commits into
UE5.6from
backport/UE5.6/pr-1156
Nov 4, 2026
Merged

[UE5.6] fix(signalling): leave the previous streamer when an SFU subscribes again (#1156)#1172
mcottontensor merged 2 commits into
UE5.6from
backport/UE5.6/pr-1156

Conversation

@mcottontensor

@mcottontensor mcottontensor commented Nov 4, 2026

Copy link
Copy Markdown
Collaborator

Backport

This will backport the following commits from master to UE5.6:

Questions ?

Please refer to the Backport tool documentation

@marekl11
fix(signalling): leave the previous streamer when an SFU subscribes a…
a8be3b3 
…gain (#1156)

subscribe() replaced subscribedStreamer without unsubscribing, so the old streamer kept forwarding to the SFU and could later detach it.

(cherry picked from commit fd0eef5)
@changeset-bot

changeset-bot Bot commented Nov 4, 2026 •
edited
Loading

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 9bb03bc

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/lib-pixelstreamingsignalling-ue5.6 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

@mcottontensor mcottontensor mentioned this pull request Nov 4, 2026
Merged
1 of 7 tasks
@mcottontensor
fix: use ue5.6 package names in backported test and changeset
9bb03bc 
The cherry-pick kept master's -ue5.8 suffix, which broke the Signalling jest suite on this branch.
@mcottontensor
mcottontensor merged commit 90c9ef8 into UE5.6 Nov 4, 2026
4 checks passed
@mcottontensor
mcottontensor deleted the backport/UE5.6/pr-1156 branch November 4, 2026 22:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mcottontensor @marekl11