-
Notifications
You must be signed in to change notification settings - Fork 233
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improvement/cldsrv 428 put api imp deny #5325
base: improvement/CLDSRV-427-permission-checks
Are you sure you want to change the base?
Improvement/cldsrv 428 put api imp deny #5325
Conversation
Hello kaztoozs,My role is to assist you with the merge of this Status report is not available. |
Branches have divergedThis pull request's source branch To avoid any integration risks, please re-synchronize them using one of the
Note: If you choose to rebase, you may have to ask me to rebuild |
117af54
to
548a28d
Compare
d81ebd9
to
cd2db8f
Compare
A significant portion of the tests have been skipped (and marked with TODOs for following tickets) due to the nature of these logic changes making the S3 APIs non-functional. These will be re-enabled in the ensuing tickets and these will likely need to be merged into this branch before making any merge into a dev branch |
914c0db
to
fe6578b
Compare
9d2567e
to
dfa80a4
Compare
A significant portion of the tests have been skipped (and marked with TODOs for following tickets) due to the nature of these logic changes making the S3 APIs non-functional. These will be re-enabled in the ensuing tickets and these will likely need to be merged into this branch before making any merge into a dev branch.
So reviewers: pay most attention to the PUT APIs and the logic changes: a large amount of files are only changed to add these skips