On Wednesday, July 22, 2020 7:49 PM, Vincent Wang <firstname.lastname@example.org> wrote:
> Thanks for the feedback! I'll implement these fixes immediately.
No rush :)
> P.S. As far as I can tell, the try..catch around new WebSocket does> not actually catch connection errors - they are instead sent to the> error event handler. Is there another reason for the try..catch> block?
According to the spec , exceptions can be triggered when the
WebSocket URL is invalid. Since this is a user-supplied value, it's
better to catch these errors.