Skip to content

feat(admin): add typed IAM policy detach#323

Merged
cxymds merged 1 commit into
mainfrom
cxymds/feat-policy-detach
Jul 24, 2026
Merged

feat(admin): add typed IAM policy detach#323
cxymds merged 1 commit into
mainfrom
cxymds/feat-policy-detach

Conversation

@cxymds

@cxymds cxymds commented Jul 24, 2026

Copy link
Copy Markdown
Member

Summary

  • add rc admin policy detach with exactly one user/group selector and deterministic multi-policy input
  • use typed, bounded admin request/response handling and fail closed when the detach capability is absent
  • report detached, still-attached, and unchanged policies with stable human and output-v3 results
  • preserve idempotent retry behavior and distinguish auth, not-found, unsupported, validation, conflict, and network failures

BREAKING contract update

The protected command reference is extended with the new detach subcommand and its selector rules. Existing commands and output schemas remain backward compatible.

Validation

  • cargo fmt --all --check
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo test --workspace
  • git diff --check

Closes rustfs/backlog#1492
Refs rustfs/backlog#1380

@cxymds
cxymds merged commit 6b2f94c into main Jul 24, 2026
17 checks passed
@cxymds
cxymds deleted the cxymds/feat-policy-detach branch July 24, 2026 13:13
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