dktr0 / estuary

Platform for collaboration and learning through live coding
GNU General Public License v3.0
165 stars 27 forks source link

use withPingThread in server #212

Closed dktr0 closed 1 year ago

dktr0 commented 2 years ago

forkPingThread is deprecated. Presumably we should use withPingThread instead. In connection with this work, we can return to the issue of adding extra trace-ing for exceptions in either the ping or main threads that may be causing problems with persistent "ghost" connections.

dktr0 commented 2 years ago

See also https://github.com/jaspervdj/websockets/issues/159