Skip to content

Fix DeepSec security findings#869

Merged
nicoalbanese merged 2 commits intomainfrom
na/445ca17b
May 5, 2026
Merged

Fix DeepSec security findings#869
nicoalbanese merged 2 commits intomainfrom
na/445ca17b

Conversation

@nicoalbanese
Copy link
Copy Markdown
Collaborator

Summary

Fixes the DeepSec report findings across agent tools and web API routes.

Changes

  • Harden agent tools against local file reads through webFetch, workspace path
    traversal, symlinked .env access, unsafe .env writes, and weak bash approval
    detection.
  • Fix hosted demo bypasses for repository-backed sessions and trial message
    limits.
  • Add shared redirect sanitization for GitHub OAuth/App flows and related client
    redirect paths.
  • Add lightweight per-user rate limiting and bot checks to expensive endpoints.
  • Require session ownership for sandbox creation and remove the orphan ephemeral
    sandbox path.
  • Mask internal error details in client responses.
  • Fix code-editor launch races with a sandbox lock.
  • Fix preference mass assignment, Vercel token revocation reporting, prompt
    snippet escaping, and trusted deployment URL extraction.
  • Add/update tests for trial message replay, sandbox creation, and code-editor
    launch locking.

Validation

  • bun run ci

@vercel
Copy link
Copy Markdown

vercel Bot commented May 5, 2026

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

Project Deployment Actions Updated (UTC)
open-agents Ready Ready Preview May 5, 2026 9:37am

Comment thread packages/agent/tools/fetch.ts Outdated
@nicoalbanese nicoalbanese merged commit c90249b into main May 5, 2026
5 checks passed
@nicoalbanese nicoalbanese deleted the na/445ca17b branch May 5, 2026 09:40
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