-
Notifications
You must be signed in to change notification settings - Fork 2
[SEC-2.1] Tool result sanitization boundary #1200
Copy link
Copy link
Closed
Labels
priority/highHigh priorityHigh prioritysecuritySecurity-related issueSecurity-related issuesize/MMedium PR (51-200 lines)Medium PR (51-200 lines)toolsTool execution and MCP integrationTool execution and MCP integration
Description
Part of #1195 — Phase 2
Apply sanitization at ToolExecutor boundary. All tool outputs must carry TrustLevel metadata.
Crates: zeph-tools, zeph-core
Depends on: SEC-1.4
Tasks:
-
ToolResultgainstrust_level: TrustLevelfield -
ShellExecutor: results taggedLocalUntrusted -
WebScrapeExecutor: results taggedExternalUntrusted, extra sanitization (HTML tag stripping) -
CompositeExecutor: propagates trust level from inner executors - Audit log:
tracing::info!for each tool result with trust level and size - Unit tests per executor type
Files: crates/zeph-tools/src/lib.rs, crates/zeph-tools/src/shell.rs, crates/zeph-tools/src/web_scrape.rs, crates/zeph-tools/src/composite.rs
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
priority/highHigh priorityHigh prioritysecuritySecurity-related issueSecurity-related issuesize/MMedium PR (51-200 lines)Medium PR (51-200 lines)toolsTool execution and MCP integrationTool execution and MCP integration