Refactor server data handling to use new API response structure and update UI components accordingly
Some checks failed
CI - Build and Push / Build and Push Docker Image (push) Failing after 14s
Some checks failed
CI - Build and Push / Build and Push Docker Image (push) Failing after 14s
This commit is contained in:
@@ -1,31 +1,27 @@
|
|||||||
import { createContext, useContext, useState, useEffect } from 'react'
|
import { createContext, useContext, useState, useEffect } from 'react'
|
||||||
import type { ReactNode } from 'react'
|
import type { ReactNode } from 'react'
|
||||||
|
|
||||||
interface ServerInfo {
|
// Response from the new /api/server/statistics/list endpoint
|
||||||
|
interface ServerListResponse {
|
||||||
|
serverName: string
|
||||||
|
serverIP: string
|
||||||
|
serverPort: number
|
||||||
|
category: string
|
||||||
|
mapName: string
|
||||||
|
playerCount: number
|
||||||
|
botCount: number
|
||||||
|
maxPlayers?: number
|
||||||
|
}
|
||||||
|
|
||||||
|
export interface ServerData {
|
||||||
name: string
|
name: string
|
||||||
ip: string
|
ip: string
|
||||||
port: number
|
port: number
|
||||||
category: 'Surf' | 'Kz' | 'Bhop'
|
category: string
|
||||||
}
|
|
||||||
|
|
||||||
interface A2SResponse {
|
|
||||||
appID: number
|
|
||||||
botCount: number
|
|
||||||
environment: string
|
|
||||||
gameDescription: string
|
|
||||||
gameDirectory: string
|
|
||||||
mapName: string
|
mapName: string
|
||||||
maxPlayers: number
|
|
||||||
playerCount: number
|
playerCount: number
|
||||||
serverName: string
|
botCount: number
|
||||||
serverType: string
|
maxPlayers: number
|
||||||
vac: number
|
|
||||||
version: string
|
|
||||||
visibility: number
|
|
||||||
}
|
|
||||||
|
|
||||||
export interface ServerData extends ServerInfo {
|
|
||||||
a2sData?: A2SResponse
|
|
||||||
status: 'online' | 'offline' | 'loading'
|
status: 'online' | 'offline' | 'loading'
|
||||||
error?: string
|
error?: string
|
||||||
}
|
}
|
||||||
@@ -39,19 +35,6 @@ interface ServerContextType {
|
|||||||
|
|
||||||
const ServerContext = createContext<ServerContextType | undefined>(undefined)
|
const ServerContext = createContext<ServerContextType | undefined>(undefined)
|
||||||
|
|
||||||
// Server list from database export
|
|
||||||
const serverList: ServerInfo[] = [
|
|
||||||
{ name: 'Surf 66 Tick #1', ip: '14.103.233.1', port: 27015, category: 'Surf' },
|
|
||||||
{ name: 'Surf 66 Tick #2', ip: '14.103.233.1', port: 27016, category: 'Surf' },
|
|
||||||
{ name: 'Surf 66 Tick #3', ip: '14.103.233.1', port: 27017, category: 'Surf' },
|
|
||||||
{ name: 'Surf 100 Tick #1', ip: '14.103.233.1', port: 28015, category: 'Surf' },
|
|
||||||
{ name: 'Surf 100 Tick #2', ip: '14.103.233.1', port: 28016, category: 'Surf' },
|
|
||||||
{ name: 'Kz #1', ip: '14.103.233.1', port: 29015, category: 'Kz' },
|
|
||||||
{ name: 'Kz #2', ip: '14.103.233.1', port: 29016, category: 'Kz' },
|
|
||||||
{ name: 'Bhop #1', ip: '14.103.233.1', port: 30015, category: 'Bhop' },
|
|
||||||
{ name: 'Bhop #2', ip: '14.103.233.1', port: 30016, category: 'Bhop' }
|
|
||||||
]
|
|
||||||
|
|
||||||
export function ServerProvider({ children }: { children: ReactNode }) {
|
export function ServerProvider({ children }: { children: ReactNode }) {
|
||||||
const [servers, setServers] = useState<ServerData[]>([])
|
const [servers, setServers] = useState<ServerData[]>([])
|
||||||
const [loading, setLoading] = useState(true)
|
const [loading, setLoading] = useState(true)
|
||||||
@@ -60,52 +43,32 @@ export function ServerProvider({ children }: { children: ReactNode }) {
|
|||||||
|
|
||||||
const fetchServerData = async () => {
|
const fetchServerData = async () => {
|
||||||
try {
|
try {
|
||||||
// Batch query all servers in a single API call
|
// Fetch server list from the new API endpoint
|
||||||
const response = await fetch('/api/server/statistics/a2s-query', {
|
const response = await fetch('/api/server/statistics/list', {
|
||||||
method: 'POST',
|
method: 'GET',
|
||||||
headers: { 'Content-Type': 'application/json' },
|
headers: { 'Content-Type': 'application/json' }
|
||||||
body: JSON.stringify({
|
|
||||||
servers: serverList.map(server => ({
|
|
||||||
serverIP: server.ip,
|
|
||||||
serverPort: server.port,
|
|
||||||
timeout: 3000
|
|
||||||
}))
|
|
||||||
})
|
|
||||||
})
|
})
|
||||||
|
|
||||||
if (!response.ok) {
|
if (!response.ok) {
|
||||||
throw new Error(`HTTP ${response.status}`)
|
throw new Error(`HTTP ${response.status}`)
|
||||||
}
|
}
|
||||||
|
|
||||||
const data = await response.json()
|
const data: ServerListResponse[] = await response.json()
|
||||||
|
|
||||||
// Update servers state with new data
|
// Transform API response to ServerData format
|
||||||
setServers(prevServers => {
|
const serverData: ServerData[] = data.map(server => ({
|
||||||
const results: ServerData[] = serverList.map((server, index) => {
|
name: server.serverName,
|
||||||
const a2sData = data.results?.[index]
|
ip: server.serverIP,
|
||||||
|
port: server.serverPort,
|
||||||
if (a2sData) {
|
category: server.category,
|
||||||
return {
|
mapName: server.mapName,
|
||||||
...server,
|
playerCount: server.playerCount,
|
||||||
a2sData,
|
botCount: server.botCount,
|
||||||
status: 'online' as const
|
maxPlayers: server.maxPlayers || 0,
|
||||||
}
|
status: 'online' as const
|
||||||
} else {
|
}))
|
||||||
// If no data for this server but we have previous data, keep the previous data
|
|
||||||
const prevServer = prevServers.find(s => s.ip === server.ip && s.port === server.port)
|
|
||||||
if (prevServer?.a2sData) {
|
|
||||||
return prevServer // Keep previous successful data
|
|
||||||
}
|
|
||||||
return {
|
|
||||||
...server,
|
|
||||||
status: 'offline' as const,
|
|
||||||
error: 'No response from server'
|
|
||||||
}
|
|
||||||
}
|
|
||||||
})
|
|
||||||
return results
|
|
||||||
})
|
|
||||||
|
|
||||||
|
setServers(serverData)
|
||||||
setLoading(false)
|
setLoading(false)
|
||||||
setLastUpdated(Date.now())
|
setLastUpdated(Date.now())
|
||||||
setHasInitialData(true)
|
setHasInitialData(true)
|
||||||
@@ -116,19 +79,14 @@ export function ServerProvider({ children }: { children: ReactNode }) {
|
|||||||
// Only show error state if this is the very first fetch attempt
|
// Only show error state if this is the very first fetch attempt
|
||||||
setServers(prevServers => {
|
setServers(prevServers => {
|
||||||
// Check if we have any previous successful data
|
// Check if we have any previous successful data
|
||||||
const hasPreviousData = prevServers.some(s => s.a2sData)
|
const hasPreviousData = prevServers.length > 0 && prevServers.some(s => s.status === 'online')
|
||||||
|
|
||||||
if (hasPreviousData) {
|
if (hasPreviousData) {
|
||||||
// Keep previous data on polling errors
|
// Keep previous data on polling errors
|
||||||
return prevServers
|
return prevServers
|
||||||
} else {
|
} else {
|
||||||
// First fetch failed, show offline state
|
// First fetch failed, return empty array
|
||||||
const offlineServers: ServerData[] = serverList.map(server => ({
|
return []
|
||||||
...server,
|
|
||||||
status: 'offline' as const,
|
|
||||||
error: error instanceof Error ? error.message : 'Unknown error'
|
|
||||||
}))
|
|
||||||
return offlineServers
|
|
||||||
}
|
}
|
||||||
})
|
})
|
||||||
|
|
||||||
|
|||||||
@@ -238,7 +238,7 @@ function Servers() {
|
|||||||
fontSize: '0.9rem',
|
fontSize: '0.9rem',
|
||||||
color: 'var(--text-secondary)'
|
color: 'var(--text-secondary)'
|
||||||
}}>
|
}}>
|
||||||
<span>🗺️ {server.a2sData?.mapName || 'Loading...'}</span>
|
<span>🗺️ {server.mapName || 'Loading...'}</span>
|
||||||
<span>•</span>
|
<span>•</span>
|
||||||
<span>{server.category}</span>
|
<span>{server.category}</span>
|
||||||
</div>
|
</div>
|
||||||
@@ -264,7 +264,7 @@ function Servers() {
|
|||||||
fontWeight: 'bold',
|
fontWeight: 'bold',
|
||||||
color: 'var(--text-primary)'
|
color: 'var(--text-primary)'
|
||||||
}}>
|
}}>
|
||||||
{server.a2sData ? `${server.a2sData.playerCount - server.a2sData.botCount}/${server.a2sData.maxPlayers}` : 'Loading...'}
|
{server.status === 'online' ? `${server.playerCount - server.botCount}/${server.maxPlayers}` : 'Loading...'}
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
<div>
|
<div>
|
||||||
@@ -380,7 +380,7 @@ function Servers() {
|
|||||||
color: 'var(--accent-primary)',
|
color: 'var(--accent-primary)',
|
||||||
marginBottom: '0.5rem'
|
marginBottom: '0.5rem'
|
||||||
}}>
|
}}>
|
||||||
{servers.reduce((sum, s) => sum + (s.a2sData ? (s.a2sData.playerCount - s.a2sData.botCount) : 0), 0)}
|
{servers.reduce((sum, s) => sum + (s.status === 'online' ? (s.playerCount - s.botCount) : 0), 0)}
|
||||||
</div>
|
</div>
|
||||||
<div style={{
|
<div style={{
|
||||||
color: 'var(--text-secondary)',
|
color: 'var(--text-secondary)',
|
||||||
|
|||||||
Reference in New Issue
Block a user