zenermerps/SocialDragon
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

Changing socket behaviour to not check for errors on read

Browse source 
This fixes #4, which crashed the server due to the endless
reading-loop and therefore the possibility for the slice
content to change in another goroutine.
This commit is contained in:
Andreas Mieke 2017-05-13 02:00:56 +02:00
parent 6b31de0620
commit bcbb789cdd
1 changed files with 0 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
socialdragon/socket.go
View file

@ -25,7 +25,6 @@ func wsHandler(w http.ResponseWriter, r *http.Request) {
log.Printf("Failed to set websocket upgrade: %+v", err) log.Printf("Failed to set websocket upgrade: %+v", err)
return return
} }
pos := len(sockets)
sockets = append(sockets, conn) sockets = append(sockets, conn)
for { for {
_, _, err := conn.ReadMessage() _, _, err := conn.ReadMessage()
@ -33,7 +32,6 @@ func wsHandler(w http.ResponseWriter, r *http.Request) {
break break
} }
} }
sockets = append(sockets[:pos], sockets[pos+1:]...)
} }
func SendUpdate(IT database.Item) { func SendUpdate(IT database.Item) {