Skip to content

fix: security hardening, admin route enforcement, and bug fixes#2

Merged
4RH1T3CT0R7 merged 1 commit intomainfrom
review/comprehensive-audit-2
Feb 15, 2026
Merged

fix: security hardening, admin route enforcement, and bug fixes#2
4RH1T3CT0R7 merged 1 commit intomainfrom
review/comprehensive-audit-2

Conversation

@4RH1T3CT0R7
Copy link
Owner

Summary

  • Path traversal prevention in program Create and Update handlers (filepath.Clean + .. rejection)
  • Admin route enforcement — tournament Create/Start/Complete/CreateMatch moved behind RequireAdmin() middleware
  • Frontend AdminRoute guard — client-side admin check for /admin route
  • SQL placeholder fix in BulkInserter for 10+ columns (fmt.Sprintf instead of rune arithmetic)
  • File size enforcement in FileStorage using io.LimitReader (defense against spoofed headers)
  • Container log OOM prevention — 2MB combined limit on Docker log output
  • Pagination caps (max 1000) on game and tournament list endpoints
  • Worker pool scale-down safety — in-flight matches use pool context, not worker context
  • WebSocket panic recoverysendPong recovers from send-on-closed-channel
  • Cache nil guard in match processor

Test plan

  • go vet ./... — clean
  • go test -race -p 1 ./internal/... ./pkg/... — 23/23 packages pass, 0 data races
  • 11 review agents confirmed no remaining issues

…trict admin routes, and improve logging, caching, and error handling
@4RH1T3CT0R7 4RH1T3CT0R7 merged commit 2925277 into main Feb 15, 2026
5 checks passed
@4RH1T3CT0R7 4RH1T3CT0R7 deleted the review/comprehensive-audit-2 branch February 15, 2026 17:49
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