zotero / stream-server

Zotero Streaming API Server
GNU Affero General Public License v3.0
9 stars 8 forks source link

Check for response from keepalive #13

Closed dstillman closed 5 years ago

dstillman commented 5 years ago

We're sending pings to the client, but we're not checking for the response. It's possible that this is causing us to not properly detect broken connections, which would presumably cause a memory leak. We should check for a pong as explained in the ws docs.