Replace pull responder listener with a websocket listener. - Githubissues

powerhouse-inc / powerhouse

https://powerhouse-design-system-six.vercel.app

0 stars 0 forks source link

Replace pull responder listener with a websocket listener. #229

Open CallmeT-ty opened 5 months ago

CallmeT-ty commented 5 months ago

Start using GraphQL websockets subscriptions for receiving pull updates from switchboard. We need to detect:

When the websocket subcript is graciously closed by the server.
When it is graciously closed by the client (page reopen)
When it times out.

Allowing connect or switchboard to pull & push whenever they want.

[ ] Implementation (should be nearly done)
[ ] Requires testing
[ ] Flow of connecting & disconnecting needs to be tested
[ ] We need to make sure that whenever a connection is closed all of the resources are freed up again. This detection of reopening the socket will help for version conflicts in Connect.

DoD

Requires multiple manual testing for the various close / reconnect scenario's (timing scenario's)
Requires load testing to determine memory usage
Pull request listener can't be deleted (back up system)

CallmeT-ty commented 4 months ago

Requires more active management of the Listeners. We'll bring this back to priority once the switchboard rework is finished.