Skip to content
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

[sos acl add] Fix Full-Control on object when adding a manual Grant rule #194

Merged
merged 11 commits into from
Dec 9, 2019

Conversation

pierre-emmanuelJ
Copy link
Member

@pierre-emmanuelJ pierre-emmanuelJ commented Nov 6, 2019

This PR fix the exo sos acl add command.
It's fix the Full-Control on object when adding a manual Grant rule, by taking the original object owner ID and not the current account ID

This PR is blocked until this PR: minio/minio-go#1182 is Merged

Fix ch-6409

@pierre-emmanuelJ pierre-emmanuelJ added the HOLD ✋ Hold, don't merge label Nov 6, 2019
cmd/sos_acl.go Outdated Show resolved Hide resolved
cmd/sos_acl.go Show resolved Hide resolved
Copy link
Contributor

@illi-j illi-j left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🐈

pierre-emmanuelJ and others added 7 commits December 9, 2019 14:24
Signed-off-by: Pierre-Emmanuel Jacquier <pierre-emmanuel.jacquier@exoscale.ch>
Signed-off-by: Pierre-Emmanuel Jacquier <pierre-emmanuel.jacquier@exoscale.ch>
Signed-off-by: Pierre-Emmanuel Jacquier <pierre-emmanuel.jacquier@exoscale.ch>
Signed-off-by: Pierre-Emmanuel Jacquier <pierre-emmanuel.jacquier@exoscale.ch>
Signed-off-by: Pierre-Emmanuel Jacquier <15922119+pierre-emmanuelJ@users.noreply.github.com>
Signed-off-by: Pierre-Emmanuel Jacquier <15922119+pierre-emmanuelJ@users.noreply.github.com>
Signed-off-by: Pierre-Emmanuel Jacquier <15922119+pierre-emmanuelJ@users.noreply.github.com>
Signed-off-by: Pierre-Emmanuel Jacquier <15922119+pierre-emmanuelJ@users.noreply.github.com>
Signed-off-by: Pierre-Emmanuel Jacquier <15922119+pierre-emmanuelJ@users.noreply.github.com>
Signed-off-by: Pierre-Emmanuel Jacquier <15922119+pierre-emmanuelJ@users.noreply.github.com>
Signed-off-by: Pierre-Emmanuel Jacquier <15922119+pierre-emmanuelJ@users.noreply.github.com>
@pierre-emmanuelJ pierre-emmanuelJ merged commit 74a1dcf into master Dec 9, 2019
@pierre-emmanuelJ pierre-emmanuelJ deleted the sos-acl branch December 9, 2019 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants