Skip to content

Comments

fix(user-pool): enforce client operations within pool scope#23

Open
unvalley wants to merge 1 commit intomainfrom
codex/main-client-scope-boundary
Open

fix(user-pool): enforce client operations within pool scope#23
unvalley wants to merge 1 commit intomainfrom
codex/main-client-scope-boundary

Conversation

@unvalley
Copy link
Owner

Summary

  • validate UserPoolId before listing or deleting pool clients
  • ensure DeleteUserPoolClient only deletes a client that belongs to the specified pool
  • add regression tests for wrong-pool and missing-pool cases

Why

DeleteUserPoolClient accepted UserPoolId but did not enforce pool ownership, allowing cross-pool deletion when a valid ClientId from another pool was supplied.

Verification

  • cargo fmt
  • cargo clippy -- -D warnings
  • cargo test

Supersedes #20 (same fix rebuilt from main).

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