Skip to content

fix(ui): show Change Password only for users with consoleAdmin#68

Merged
overtrue merged 1 commit intorustfs:mainfrom
GatewayJ:main
Feb 28, 2026
Merged

fix(ui): show Change Password only for users with consoleAdmin#68
overtrue merged 1 commit intorustfs:mainfrom
GatewayJ:main

Conversation

@GatewayJ
Copy link
Member

Align with RustFS backend: only users with consoleAdmin policy can change password (see rustfs/rustfs#1923). Add canChangePassword to permissions (isAdmin or has consoleAdmin scope) and show the menu item only when true, so non-admin users no longer see the option and get 403 on submit.

Made-with: Cursor

Pull Request

Description

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Test improvements
  • Security fix

Testing

  • Unit tests added/updated
  • Manual testing completed
pnpm test:run

Checklist

  • Code follows the project's style guidelines
  • Self-review completed
  • TypeScript types are properly defined
  • All commit messages are in English (Conventional Commits)
  • All existing tests pass
  • No new dependencies added, or they are justified

Related Issues

Closes #

Screenshots (if applicable)

Additional Notes

Align with RustFS backend: only users with consoleAdmin policy can change
password (see rustfs/rustfs#1923). Add canChangePassword to permissions
(isAdmin or has consoleAdmin scope) and show the menu item only when true,
so non-admin users no longer see the option and get 403 on submit.

Made-with: Cursor
@GatewayJ GatewayJ requested a review from overtrue February 28, 2026 03:41
@overtrue overtrue merged commit 96dc73b into rustfs:main Feb 28, 2026
11 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.

2 participants