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)
Sat, Nov 23, 12:04 PM
Unknown Object (File)
Sat, Nov 23, 11:56 AM
Unknown Object (File)
Sat, Nov 23, 9:19 AM
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
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
No Lint Coverage
Unit
No Test Coverage