fix: players list not properly calculated

Signed-off-by: Neko Ayaka <neko@ayaka.moe>

Author: nekomeowww

README.md

@@ -8,8 +8,8 @@
 
 | Factorio RCON API | Factorio Server |
 |-------------------|-----------------|
-| 1.1.0 (`/api/v1`) | 1.x             |
-| 2.0.0 (`/api/v2`) | 2.x             |
+| `/api/v1`         | 1.x             |
+| `/api/v2`         | 2.x             |
 
 ## Features
 

pkg/utils/factorio.go

@@ -20,14 +20,18 @@ func StringListToPlayers(list string) ([]*v1.Player, error) {
 	for _, line := range split {
 		line = strings.TrimSpace(line)
 		parts := strings.Split(line, " ")
-		if len(parts) != 2 {
+		if len(parts) > 2 {
 			return nil, apierrors.NewErrBadRequest().WithDetailf("failed to parse admins: %s due to parts not equals 2", line).AsStatus()
 		}
 
-		players = append(players, &v1.Player{
+		player := &v1.Player{
 			Username: parts[0],
-			Online:   parts[1] == "(online)",
-		})
+		}
+		if len(parts) == 2 {
+			player.Online = parts[1] == "(online)"
+		}
+
+		players = append(players, player)
 	}
 
 	return players, nil

pkg/utils/factorio_test.go

@@ -7,7 +7,7 @@ import (
 )
 
 func TestStringListToPlayers(t *testing.T) {
-	players, err := StringListToPlayers("  NekoMeow (online)\n  NekoMeow2 (offline)\n")
+	players, err := StringListToPlayers("  NekoMeow (online)\n  NekoMeow2\n")
 	require.NoError(t, err)
 	require.Len(t, players, 2)
 }