Page MenuHomePhabricator

[keyserver] Track closed state in TunnelbrokerSocket
ClosedPublic

Authored by ashoat on Mar 26 2024, 8:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 12, 11:18 PM
Unknown Object (File)
Tue, Nov 12, 11:18 PM
Unknown Object (File)
Tue, Nov 12, 11:18 PM
Unknown Object (File)
Tue, Nov 12, 9:39 PM
Unknown Object (File)
Fri, Nov 1, 9:54 PM
Unknown Object (File)
Fri, Nov 1, 9:54 PM
Unknown Object (File)
Fri, Nov 1, 9:54 PM
Unknown Object (File)
Fri, Nov 1, 9:36 PM
Subscribers

Details

Summary

This diff makes sure we don't respond to 'open' and 'close' events that come after a 'close' event. It mirrors code in lib/socket/socket.react.js.

We need this additional state so that in the on('open' block, we can differentiate the state prior to initialization to the state after closure.

Test Plan

I'm actually not sure how to reproduce an open or close event after a close event. Just adding this code because it was in the other class, and I don't think it will hurt

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable