]> git.proxmox.com Git - mirror_novnc.git/commit
Fixed a race condition when attaching to an existing socket
authorTim Stableford <tims@bsquare.com>
Tue, 23 Mar 2021 12:58:49 +0000 (12:58 +0000)
committerTim Stableford <tims@bsquare.com>
Tue, 30 Mar 2021 13:32:04 +0000 (14:32 +0100)
commitef27628c6dff6120b0ed0d4728cc6e8a32b7be53
tree0b6eff930ae6d6307f9be739dc546bf6ab1acaab
parent89f9ac00166f1106e03d46562ffeaa3d8032f399
Fixed a race condition when attaching to an existing socket

This is an error that presents itself with RTCDataChannel's, I suspect this could not
happen with a pre-existing WebSocket.

If the remote connection creates a data channel then the local (VNC) side gets a channel
created callback. It may also be the case that in that very same tick the socket is also
opened and buffered data received. This meant that (in my tests) about 1/3 of the time
noVNC would fail to respond to the initial message from the server because it was received
and subsequently not handled during that initial tick.

Also made the documentation reflect this new behaviour and document the existing behaviour.
core/rfb.js
docs/API.md