search

univrsal / input-overlay

Show keyboard, gamepad and mouse input on stream
GNU General Public License v2.0
2.91k stars 246 forks source link

Remove web socket from list on close message #313

Closed katzenbar closed 1 year ago

katzenbar commented 1 year ago

The web socket was not getting marked as draining or closing when doing certain actions (closing OBS, refreshing the browser view, changing scenes), and resulted in a crash when continuing to send messages to the socket.

Fixes #230

univrsal commented 1 year ago

I guess this part can be removed then https://github.com/univrsal/input-overlay/blob/net-cleanup/src/network/mg.cpp#L66-L68

katzenbar commented 1 year ago

@univrsal Good call, updated.

univrsal commented 1 year ago

Thanks for the PR