refactor(permissions): define default permission set - Githubissues

hyperledger-iroha / iroha

Iroha - A simple, enterprise-grade decentralized ledger

https://wiki.hyperledger.org/display/iroha

Apache License 2.0

438 stars 280 forks source link

refactor(permissions): define default permission set #5075

Closed mversic closed 4 weeks ago

mversic commented 1 month ago

Context

added CanManagePeers, CanRegisterDomain and CanManageRoles permissions
unified CanSetKeyValueXXX + CanRemoveKeyValueXXX = CanModifyXXXMetadata
unified CanMintAsset + CanBurnAsset + CanTransferAsset = CanModifyAsset
changed FindPermissions so it returns only inherent permissions, not from roles
fixed validation of grant/revoke for roles

closes #4206

Solution

Describe the approach taken to achieve the objective / resolve the issue.

Review notes (optional)

For complex PRs, try to provide some information on how to approach the review more effectively.
For example, is there a natural order in which the affected files should be reviewed?

Checklist

[ ] I've read CONTRIBUTING.md.
[ ] (optional) I've written unit tests for the code changes.
[ ] All review comments have been resolved.
[ ] All CI checks pass.

github-actions[bot] commented 1 month ago

@BAStos525