refactor: update reconnect code for better understanding

internal/websocket/connect.go

@@ -1,9 +1,10 @@
 package websocket
 
 import (
-	"github.com/gorilla/websocket"
 	"time"
 
+	"github.com/gorilla/websocket"
+
 	ChronoTomato "git.smsvc.net/pomodoro/ChronoTomato/pkg/models"
 )
 

internal/websocket/receive.go

@@ -22,27 +22,22 @@ func (c *Client) ProcessServerMessages(channel chan<- GoTomato.ServerMessage) {
 		c.Conn.SetReadDeadline(time.Now().Add(10 * time.Second))
 		_, message, err := c.Conn.ReadMessage()
 		if err != nil {
+			c.LastErr = err
+
 			// On normal closure exit gracefully
 			if websocket.IsCloseError(err, websocket.CloseNormalClosure) {
 				return
 			}
 
-			c.LastErr = err
+			// reset connection and reconnect
-			// Try to reconnect on unexpected disconnect
+			c.Conn = nil
-			for {
-				channel <- serverMessage // send last known ServerMessage to update view
-
-				time.Sleep(time.Second)
-
-				// reconnect while preserving password
 			pw := c.Password
+			for !c.Connected() {
+				channel <- serverMessage // send last known ServerMessage to update view
+				time.Sleep(time.Second)
 				*c = Connect(c.Server)
+			}
 			c.Password = pw
-
-				if c.Connected() {
-					break
-				}
-			}
 			continue
 		}