Skip to content

Harden daemon trust boundaries - #118

Merged
pquerna merged 1 commit into
mainfrom
paul.querna/harden-daemon-trust-boundary
Jun 29, 2026
Merged

Harden daemon trust boundaries#118
pquerna merged 1 commit into
mainfrom
paul.querna/harden-daemon-trust-boundary

Conversation

@c1-squire-dev

@c1-squire-dev c1-squire-dev Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • propagate --allowed-binaries into lazily spawned daemons from CLI and MCP entry points
  • tighten executable allowlist semantics so bare names only match bare argv[0], while path-style invocations require exact canonical path entries
  • gate daemon upgrade through the same executable governance path as spawn
  • enforce user-owned 0700 daemon socket roots on Unix
  • bound daemon and MCP JSON-RPC line frames to 1 MiB
  • restrict PNG snapshot writes to new relative files under the daemon cwd, rejecting absolute/traversal paths and overwrites
  • add focused regressions for lazy-spawn governance, allowlist bypasses, socket permissions, bounded frames, and PNG path policy

Testing

  • cargo fmt --all -- --check
  • RUSTFLAGS='-D warnings' cargo test --workspace
  • RUSTFLAGS='-D warnings' cargo clippy --workspace --all-targets -- -D warnings
  • focused rerun: RUSTFLAGS='-D warnings' cargo test -p agent-tui-daemon --test round_trip snapshot_png -- --nocapture

@c1-squire-dev
c1-squire-dev Bot force-pushed the paul.querna/harden-daemon-trust-boundary branch 2 times, most recently from 2423f78 to 4c6d883 Compare June 29, 2026 20:35
Propagate lazy-spawn governance, tighten executable allowlist matching, bound request frames, restrict socket and PNG filesystem surfaces, and add focused regressions.

Co-authored-by: c1-squire-dev[bot] <c1-squire-dev[bot]@users.noreply.github.com>
@c1-squire-dev
c1-squire-dev Bot force-pushed the paul.querna/harden-daemon-trust-boundary branch from 4c6d883 to c36d880 Compare June 29, 2026 20:38
@pquerna
pquerna merged commit 9080152 into main Jun 29, 2026
17 checks passed
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