WebSocketServer has some oversights

The first problem is that when using a server, the listener is not always the IP! It can also be the hostname. This is a rare case, but it’s worth considering.

The second problem is that VVVV MUST be run as administrator to work with the external network (to connect to this server from an external computer). But because of the Try region, exceptions are no longer visible and finding the cause becomes a problem. It should either be removed or explicitly written to the log. It is also a good idea to include a comment in the patch about running as administrator.

thanks for pointing these out. VL.IO.WebSocket 1.1.0 comes as a minor breaking change with the input now being named “Hostname or IP” and the try region removed, allowing for issues to be caught by user code.

In case of exceptions, there is not enough information. I encountered a problem when using Webserver, I enter the hostname of the device and the node returns the exception:

“HttpListenerException
Access denied.”

The exception does not have enough information. It would be nice to expand the exception with an explanation that the application must be run with administrator roles.

@sunrise might be worth redirecting to the original repository. The author is very responsive and aware of the existence of VVVV. Although judging by the exception that occurs, it might make sense to add something inside the patch. What is definitely worth adding is text that explicitly states this (both in the node patch and in the help patch by comments)