Skip to content

Fix admin request crashes, secure defaults, and LAN player identities - #110

Merged
JonasFocus merged 1 commit into
mainfrom
fix/review-admin-and-lan-identity
Nov 9, 2026
Merged

JonasFocus merged 1 commit into
mainfrom
fix/review-admin-and-lan-identity

Conversation

@JonasFocus

Copy link
Copy Markdown
Owner

Admin event requests with fractional limits could terminate the game server and lose all in-memory rooms. Validate limits and contain unexpected HTTP handler failures, and disable admin API routes unless an explicit nonblank ADMIN_CODE is configured.

Generate cryptographic player identities on HTTP LAN origins where crypto.randomUUID is unavailable, and use deterministic opening cards for the fresh-shoe animation test to avoid random dealer-blackjack failures. Add regression coverage and document the admin configuration requirement.

Validation: all 441 tests pass; typecheck, lint, production build, and git diff --check pass.

@vercel

vercel Bot commented Nov 9, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
game-time Ready Ready Preview Nov 9, 2026 2:01am UTC

@JonasFocus
JonasFocus merged commit 0910179 into main Nov 9, 2026
3 checks passed

This branch was successfully deployed

1 active deployment
Preview — 4414b2df Deployed Nov 9, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant