whbing commented on PR #4559:
URL: https://github.com/apache/ozone/pull/4559#issuecomment-1520548538
Thacks for review.
The changes before and after this patch as follows:
| now version | check acl on | work on |
|------------------------|--------------|---------|
| bucket delete link | link | link |
| bucket info link | link | link |
| bucket getacl link | link | link |
| bucket addacl/setacl/removeacl link | link | link |
| key getacl/addacl/setacl/removeacl link | KEY_NOT_FOUND | KEY_NOT_FOUND
|
| with pr version | check acl on | work on |
|------------------------|--------------|---------|
| bucket delete link | link | link |
| bucket info link | source (link if source not exist) |
link |
| bucket getacl link | link | link |
| bucket addacl/setacl/removeacl link | source | source |
| key getacl/addacl/setacl/removeacl link | source | source |
Note:
1. `bucket info link` requires `READ`, which will be validated at the parent
level of key operation. So check on source bucket acl.
2. cli add option for `bucket getacl link`, when get source acl, will check
ACL: `link's READ` and `source's READ_ACL`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
