smengcl commented on PR #11598: URL: https://github.com/apache/ozone/pull/11598#issuecomment-6326438328
Thanks @SaketaChalamchala. Updated in 11ccf5e: - `snapshotDiff` and `submitSnapshotDiff` check READ and LIST on the bucket. With LIST only, a diff was allowed while `getSnapshotInfo` was denied. - Bucket root `listStatus`, and `listStatusLight` with partial prefixes, check LIST in addition to the existing READ check. - `testSnapshotPermissions` covers READ only, LIST only, and READ and LIST, for bucket info, snapshot info, cancel, get and submit diff, `listSnapshot`, `listSnapshotDiffJobs` and listing the bucket root. `listKeys`, `listSnapshot` and `listSnapshotDiffJobs` still check only LIST. Adding READ to them changes three more RPCs, so I suggest a separate JIRA for that. -- 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]
