KeshavAnandCode/buzzer-site
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Server: assign sequential numeric IDs (1-2-3...) to players on join instead of random strings

Browse Source
This commit is contained in:
Keshav Anand
2026-04-08 18:53:24 -05:00
parent b76bcbffb2
commit c00db744c5
1 changed files with 9 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
src/ws-handler.ts
View File

@@ -65,16 +65,19 @@ export function handleMessage(ws: WS, raw: string) {
if (!room) { er(ws, "Room not found"); return; } if (!room) { er(ws, "Room not found"); return; }
if (room.locked) { er(ws, "Room is locked"); return; } if (room.locked) { er(ws, "Room is locked"); return; }
const name = sanitize(msg.playerName ?? "Player", 24) || "Player"; // Calculate numeric ID based on join order
const existingId = sanitize(msg.playerId ?? "", 12); const joinCount = room.players.size + 1;
const playerId = joinCount.toString();
const name = sanitize(msg.playerName ?? "", 24) || ""; // Keep name for compatibility but don't use it
let player: Player | undefined; let player: Player | undefined;
if (existingId && room.players.has(existingId)) { if (playerId && room.players.has(playerId)) {
player = room.players.get(existingId)!; player = room.players.get(playerId)!;
player.ws = ws; player.isConnected = true; player.name = name; player.ws = ws; player.isConnected = true; player.name = name;
} else { } else {
player = { id: genId(), name, teamIndex: null, ws, isConnected: true }; player = { id: playerId, name, teamIndex: null, ws, isConnected: true };
room.players.set(player.id, player); room.players.set(playerId, player);
} }
wsToPlayer.set(ws, { roomId: room.id, playerId: player.id }); wsToPlayer.set(ws, { roomId: room.id, playerId: player.id });