Skip to content

Report exec policy refusals honestly - #180

Merged
myobie merged 1 commit into
mainfrom
fix/doctor-exec-refusal
Sep 5, 2026
Merged

Report exec policy refusals honestly#180
myobie merged 1 commit into
mainfrom
fix/doctor-exec-refusal

Conversation

@myobie

@myobie myobie commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator

Result

  • fabric exec now reports every pre-command failure instead of returning silent status 1.
  • ACL and machine policy refusals name the peer and exec service and return status 126.
  • fabric doctor reports an intentional exec policy refusal as info, with no attention count.
  • Unexpected version command failures remain unknown and return status 3.
  • Doctor still says an unchecked old peer can force whole-manifest fallback.

Tests

  • cargo test --locked --lib
  • cargo test --locked --test doctor
  • cargo test --locked --test local_slice exec_names_the_peer_and_service_when_the_peer_acl_refuses_it
  • cargo test --locked --test shell trusted_peer_without_allow_shell_is_refused
  • git diff --check

@myobie
myobie force-pushed the fix/doctor-exec-refusal branch from af7256b to f15d795 Compare September 5, 2026 16:24
@myobie
myobie merged commit 382c6af into main Sep 5, 2026
3 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