Skip to content

Conversation

@GatewayJ
Copy link
Contributor

@GatewayJ GatewayJ commented Jan 6, 2026

Type of Change

  • New Feature
  • Bug Fix
  • Documentation
  • Performance Improvement
  • Test/CI
  • Refactor
  • Other:

Related Issues

Summary of Changes

Checklist

  • I have read and followed the CONTRIBUTING.md guidelines
  • Passed make pre-commit
  • Added/updated necessary tests
  • Documentation updated (if needed)
  • CI/CD passed (if applicable)

Impact

  • Breaking change (compatibility)
  • Requires doc/config/deployment update
  • Other impact:

Additional Notes


Thank you for your contribution! Please ensure your PR follows the community standards (CODE_OF_CONDUCT.md) and sign the CLA if this is your first contribution.

@GatewayJ GatewayJ changed the title fix: import policy fix: Correct export permissions Jan 6, 2026
@GatewayJ GatewayJ changed the title fix: Correct export permissions fix: Correct import permissions Jan 6, 2026
@loverustfs loverustfs requested a review from Copilot January 6, 2026 06:44
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes an authorization bug in the ImportIAM operation where it was incorrectly checking for export permissions instead of import permissions.

  • Corrects the permission check in ImportIam::call from ExportIAMAction to ImportIAMAction

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@loverustfs loverustfs merged commit b95bee6 into rustfs:main Jan 6, 2026
21 of 22 checks passed
weisd pushed a commit that referenced this pull request Jan 7, 2026
houseme added a commit that referenced this pull request Jan 7, 2026
* 'main' of github.com:rustfs/rustfs:
  fix: Correct import permissions (#1402)
reatang pushed a commit that referenced this pull request Jan 11, 2026
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