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
Update get/setBucketPolicy methods to use files instead of pkg/policy #959
Conversation
2296d57
to
f4d5fca
Compare
Update |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nitisht policy as a string is simpler API than stream, that is how it is done in JS and Py
@nitisht, second @krishnasrinivas suggestion on using policy as a string or passed in as a file. |
Also removed ListBucketPolicies API as we move to json file based policy handling. Fixes minio#943
updated the PR @harshavardhana @poornas @krishnasrinivas PTAL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
@krishnasrinivas could you pls take a look again |
Also removed
ListBucketPolicies
API as we move to json file basedpolicy handling.
Fixes #943