Skip to content

fix(security): validate admin event inputs - #83

Draft
pushkart2 wants to merge 1 commit into
mainfrom
agent/harden-admin-event-inputs
Draft

fix(security): validate admin event inputs#83
pushkart2 wants to merge 1 commit into
mainfrom
agent/harden-admin-event-inputs

Conversation

@pushkart2

Copy link
Copy Markdown

Summary

  • validate client-selected admin actions and target players before dispatch
  • perform permission checks before radio export and player lookups
  • constrain report replies, ban reasons, routing buckets, radio frequencies, weapon groups, and permission names
  • route report replies using server-owned report data instead of client-supplied target data
  • safely handle players disconnecting during callbacks

Validation

  • direct Lua 5.4 syntax/control-flow review
  • git diff --check

The resource lint workflow is expected to run on this draft PR; any reported lint violations will be fixed on this branch.

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