Skip to content
This repository has been archived by the owner on Mar 1, 2024. It is now read-only.

[BUG] - MaxPlayerCount can result in a reconnection loop. #401

Closed
timbotimbo opened this issue Oct 19, 2023 · 0 comments · Fixed by #419
Closed

[BUG] - MaxPlayerCount can result in a reconnection loop. #401

timbotimbo opened this issue Oct 19, 2023 · 0 comments · Fixed by #419
Labels
bug Something isn't working

Comments

@timbotimbo
Copy link
Contributor

timbotimbo commented Oct 19, 2023

UE Version:
Tested with 5.2, but not relevant.

Frontend Version:
5.2, 5.3, 5.4
Master branch right now (current commit 2a21ee6)

(I tested 5.2 and master, so I assume 5.3 and 5.4 are the same)

Problem component
Frontend library.

Description
The frontend will endlessly try to reconnect if the SignallingServer has reached the player limit and there is no Matchmaker configured.

Using the setup:

  • 1 unreal instance
  • 1 SignallingServer without Matchmaker.
  • MaxPlayerCount >0
  • Multiple browser windows.

Steps to Reproduce:

  1. Download the repo and run the setup scripts for the SignallingServer.
  2. Run any unreal project with Pixelstreaming.
  3. Edit SignallingWebServer/config.json to set "MaxPlayerCount": 1
  4. Start the server using Start_SignallingServer.ps1.
  5. Visit the ip of the SignallingServer to reach the player.html example and start the connection. (this one works as expected)
  6. Open a second browser window doing the same.
  7. The UI in the second window will now be stuck on Auto connecting now.

The WaitForStreamer and MaxReconnectAttempts config options don't seem to make a difference either.

Expected behavior
The frontend should show a connection error or detect that it can stop trying to reconnect.

Screenshots
The browser console keeps repeating these logs.
browser

The SignallingServer keeps outputting this error.
server

Platform (please complete the following information):

  • OS: Windows 10 Pro.
  • FireFox & Chrome, any version
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Development

Successfully merging a pull request may close this issue.

1 participant