New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature+Bugfix: Engine websocket management #360
Feature+Bugfix: Engine websocket management #360
Commits on Oct 1, 2019
-
Initial commit tearing down the websocket connection management. The …
…purpose is to remove the traffic monitoring and dropping as syncer.go is a better manager
-
Adds a readwrite mutex and helper functions to minimise inline lock/u…
…nlocks and prevent races
-
Creates new WebsocketType struct to contain all parameters required. …
…Deletes WebsocketReset. Utilises ReadMessageErrors channel for all websocket readmessages to analyse when an error returned is due to a disconnect
-
-
Simplifies initialisation function for websocket. Reconnects and resu…
…bscribes after disconnection
-
Adds WebsocketTimeout config value to dictate when the websocket traf…
…fic monitor should die. Default to two minutes of no traffic activity. Increases test coverage and updates existing tests to work with new technologic. RE-ADDS TESTS I ACCIDENTALLY DELETED FROM PREVIOUS PR
-
Removes snapshot override as its always necessary when considering re…
…connections. Increases test coverage. Re-adds tests that were ACCIDENTALLY DELETED. Removes unused websocket channels. Bug fix for traffic monitor to shutdown via goroutine instead of killing itself
-
Fixes gateio bug for authentication errors when null. Adds little ent…
…ry to syncer for when websocket is switched to rest and then back, you get a log notifying of the return. Fixes okgroup bug where ws message is sent on a disconnected ws, causing panic. Renames setConnectionStatus to setConnectedStatus. Puts connection monitor log behind verbose bool
-
Fixes lingering races. Fixes bug where websocket was enabled whether …
…you liked it or not. Removes demonstration test
-
-
-
Removes go routine causing CPU spike. Stops timers properly and reset…
…s timers properly
-
Renames
WsEnabled
toEnabled
. Increases test coverage. Fixes typo……s. Handles unhandled errors
-
-
With using RWlocks, removes the channel nil check and relies on !w.Is…
…Connected() to prevent a shutdown from recurring
-
Prevents timer channel hangups. Minimises use of websocket Connect().…
… Expands disconnection error definition. Removes routine disconnection error handling. Ensures only one traffic monitor can ever be run. Renames subscriptionLock to subscriptionMutext for consistency
-
Extends timeout to 30 seconds to cover for non-popular exchanges and …
…non-popular currencies
-