Spaces:
Running
Running
Commit ·
d012f99
1
Parent(s): 4ec4472
fix: Replace ineffective break with return in WebSocket close handler
Browse filesSA4011 staticcheck warning - switch cases break implicitly in Go,
so break was a no-op. Using return to properly exit handleWebSocket
when client sends close message.
main.go
CHANGED
|
@@ -516,7 +516,7 @@ func (s *Server) handleWebSocket(w http.ResponseWriter, r *http.Request) {
|
|
| 516 |
case "state":
|
| 517 |
s.handleWSState(conn)
|
| 518 |
case "close":
|
| 519 |
-
|
| 520 |
default:
|
| 521 |
errMsg, _ := json.Marshal(map[string]string{"error": "unknown message type: " + msg.Type})
|
| 522 |
conn.WriteMessage(websocket.TextMessage, errMsg)
|
|
|
|
| 516 |
case "state":
|
| 517 |
s.handleWSState(conn)
|
| 518 |
case "close":
|
| 519 |
+
return
|
| 520 |
default:
|
| 521 |
errMsg, _ := json.Marshal(map[string]string{"error": "unknown message type: " + msg.Type})
|
| 522 |
conn.WriteMessage(websocket.TextMessage, errMsg)
|