Skip to content

security: fix critical and high issues from code audit#15

Open
wydrox wants to merge 1 commit intomainfrom
feat/8li-313-security-review
Open

security: fix critical and high issues from code audit#15
wydrox wants to merge 1 commit intomainfrom
feat/8li-313-security-review

Conversation

@wydrox
Copy link
Contributor

@wydrox wydrox commented Mar 26, 2026

Summary

  • Critical: Remove debug JSONL logging to world-readable /tmp files
  • High: Make CORS origins configurable (default localhost only)
  • High: Add request body size limits (10MB default)
  • High: Cap max_tokens to prevent memory exhaustion
  • High: Validate model names against path traversal
  • High: Sanitize VLM image URLs against SSRF
  • Full audit document in SECURITY_AUDIT.md (4-tier: Critical/High/Medium/Low)
  • 175 tests pass

Addresses 8LI-313.

🤖 Generated with Claude Code

Remove debug JSONL logging to /tmp, make CORS origins configurable,
add request body size limits, cap max_tokens, validate model names
against path traversal, sanitize VLM image URLs against SSRF.
Full audit in SECURITY_AUDIT.md.

Addresses 8LI-313.

Co-Authored-By: Claude Opus 4.6 (1M context) <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.

1 participant