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

Make the Websocket disconnect: 1006 error more informative for end users #184

Closed
3 of 6 tasks
sarahkrivan opened this issue Mar 31, 2023 · 1 comment
Closed
3 of 6 tasks
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@sarahkrivan
Copy link

Component your feature request relates to

  • Signalling server
  • Frontend library
  • Frontend UI library
  • Matchmaker
  • SFU
  • This repo

Problem your feature solves? Please describe.
Currently, all websocket disconnects are presented with the same "Websocket disconnected: 1006" error (see screenshot). This is a generic error, but not overly useful to assist with debugging whether UE disconnected or the browser disconnected.

Describe the solution you'd like
Ideally, rather than a catch all disconnect message, a user would be able to see which component has disconnected: e.g. UE disconnected, Signalling Server disconnected, Browser disconnected etc.

Additional context
websocket-disconnected

@sarahkrivan sarahkrivan added the enhancement New feature or request label Mar 31, 2023
@lukehb lukehb added the good first issue Good for newcomers label Mar 31, 2023
@EpicGames EpicGames deleted a comment Apr 27, 2023
@github-actions
Copy link
Contributor

Issues go stale after 30 days of inactivity. Please comment or re-open the issue if you are still interested in getting this issue fixed.

alien299 added a commit to alien299/PixelStreamingInfrastructure that referenced this issue Nov 28, 2023
Fixed an Issue regarding the Websocket disconnect error messages. I made them more informative so they can assist with debugging.
[Issue EpicGames#184](EpicGames#184)
alien299 added a commit to alien299/PixelStreamingInfrastructure that referenced this issue Nov 30, 2023
Fixed an Issue regarding the Websocket disconnect error messages. I made them more informative so they can assist with debugging.
[Issue EpicGames#184](EpicGames#184)
wizard0918 pushed a commit to wizard0918/Epic-Games-PixelStreamingInfrastructure that referenced this issue Apr 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request good first issue Good for newcomers
Development

No branches or pull requests

3 participants