d0zingcat/gocryptotrader
1
0
Fork 0
mirror of https://github.com/d0zingcat/gocryptotrader.git synced 2026-05-13 23:16:45 +00:00
Code Issues Packages Projects Releases Wiki Activity

Websockets: Fix subscription failure on reconnect (#1457)

Browse Source 
* Websockets: Fix subscription failure on reconnect

If a websocket is disconnected then the subscription map was left with
old subscriptions, causing:
`Okx websocket: subscription failure, channel already subscribed for ...`

* fixup! Websockets: Fix subscription failure on reconnect

Fixes review comment https://github.com/thrasher-corp/gocryptotrader/pull/1457#discussion_r1468947509
This commit is contained in:
Gareth Kirwan
2024-01-30 02:34:13 +01:00
committed by GitHub
parent 2016bfca84
commit 682737f368
1 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
exchanges/stream/websocket.go
View File

@@ -270,6 +270,10 @@ func (w *Websocket) Connect() error {
w.exchangeName)
}
w.subscriptionMutex.Lock()
w.subscriptions = subscriptionMap{}
w.subscriptionMutex.Unlock()
w.dataMonitor()
w.trafficMonitor()
w.setConnectingStatus(true)