Skip to content

fix(security): validate WHIP/WHEP OPTIONS path params - #300

Open
birme wants to merge 1 commit into
mainfrom
security/249-whip-whep-options-params-schema
Open

fix(security): validate WHIP/WHEP OPTIONS path params#300
birme wants to merge 1 commit into
mainfrom
security/249-whip-whep-options-params-schema

Conversation

@birme

@birme birme commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add a TypeBox params schema to the WHIP OPTIONS handler (/whip/:productionId/:lineId) validating productionId and lineId (String, minLength 1, maxLength 200).
  • Add the same params schema to the WHEP OPTIONS handler (/whep/:productionId/:lineId).
  • Existing response schemas are preserved; no other handlers touched.

Test plan

  • npm run typecheck clean
  • npm test all pass (243 tests; "worker process has failed to exit gracefully" warning is expected)

Closes #249

…hema

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
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.

Security: WHIP/WHEP OPTIONS handlers missing TypeBox params schema — path params unvalidated

2 participants