fix: fixing data race
This commit is contained in:
@@ -260,18 +260,24 @@ func (wsClient *SafeWebsocketClient) connect() error {
|
||||
return nil
|
||||
})
|
||||
|
||||
var c *websocket.Conn
|
||||
wsClient.mu.ReadHandler(func() error {
|
||||
c = conn
|
||||
return nil
|
||||
})
|
||||
|
||||
for {
|
||||
select {
|
||||
case <-ctx.Done():
|
||||
log.Println("Writer stopped due to client shutdown")
|
||||
return
|
||||
case data := <-wsClient.writeChan:
|
||||
if conn == nil {
|
||||
if c == nil {
|
||||
wsClient.triggerReconnect()
|
||||
return
|
||||
}
|
||||
|
||||
if err := conn.WriteMessage(int(data.MessageType), data.Data); err != nil {
|
||||
if err := c.WriteMessage(int(data.MessageType), data.Data); err != nil {
|
||||
log.Printf("error on write message: %v\n", err)
|
||||
wsClient.triggerReconnect()
|
||||
return
|
||||
@@ -287,22 +293,28 @@ func (wsClient *SafeWebsocketClient) connect() error {
|
||||
return nil
|
||||
})
|
||||
|
||||
var c *websocket.Conn
|
||||
|
||||
wsClient.mu.ReadHandler(func() error {
|
||||
c = conn
|
||||
return nil
|
||||
})
|
||||
if c == nil {
|
||||
wsClient.triggerReconnect()
|
||||
return
|
||||
}
|
||||
for {
|
||||
select {
|
||||
case <-ctx.Done():
|
||||
log.Println("Reader stopped due to client shutdown")
|
||||
return
|
||||
default:
|
||||
if conn == nil {
|
||||
wsClient.triggerReconnect()
|
||||
return
|
||||
}
|
||||
if err := conn.SetReadDeadline(time.Now().Add(readDeadline)); err != nil {
|
||||
if err := c.SetReadDeadline(time.Now().Add(readDeadline)); err != nil {
|
||||
log.Printf("error on read deadline: %v\n", err)
|
||||
return
|
||||
}
|
||||
|
||||
messageType, data, err := conn.ReadMessage()
|
||||
messageType, data, err := c.ReadMessage()
|
||||
if err != nil {
|
||||
log.Printf("error on read message: %v\n", err)
|
||||
wsClient.triggerReconnect()
|
||||
|
||||
Reference in New Issue
Block a user