Update cirrus.js - More informative error messages - Fixes EpicGames/PixelStreamingInfrastructure#184

[alien299]

Relevant components:

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

Problem statement:

Fixed an Issue regarding the WebSocket disconnect error messages. I made them more informative so they can assist with debugging. Issue #184
"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."

Solution

On abnormal closure the cirrus.js called ws.close(1006, error). I insterted more information/explanation about the error and then added the error at the end of the message. Example: ws.close(1006, "SFU connection error: ${error}");
It makes the default error message more informative as described in Issue #184
Affected Lines:
Line 662
Line 826
Line 954

Documentation

Test Plan and Compatibility

[alien299]

Hi!
@mcottontensor Should I reopen the pull request with signed commits?
The contributing guidelines didn't contain all commits have to be Verified. I fixed it in the latest 2 commits, but the original commit stayed unverified.

[mcottontensor]

It's no problem. We've just been busy lately and haven't gotten around to some things.