Skip to content

Switch to websockets#133

Draft
Protonull wants to merge 8 commits intoCivPlatform:mainfrom
Protonull:websockets
Draft

Switch to websockets#133
Protonull wants to merge 8 commits intoCivPlatform:mainfrom
Protonull:websockets

Conversation

@Protonull
Copy link
Copy Markdown
Contributor

This PR intends to replace the TCP connection with a websocket connection, which has various benefits like deferring framing to the websocket protocol, deferring encryption to TLS, etc.

@Protonull
Copy link
Copy Markdown
Contributor Author

In terms of progress, I have been focusing on the client, but I have been battling perfectionism and scope creep. As you can see, the changes also include a refactor to the authentication process. But I have stashes that include changes to the config and the connection gui to better enforce contracts (eg: that sync addresses MUST be prefixed with ws:// or wss://). I'm also snagging on how to do state checking, reconnection, etc, thread-safely. Any help here would be appreciated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants