Fix unshare for EOS storage driver (cs3org#3794)

gmgigi96 · Jun 28, 2023 · d5aa80d · d5aa80d
1 parent 9a2f1ed
commit d5aa80d
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/changelog/unreleased/fix-unshare-eos.md b/changelog/unreleased/fix-unshare-eos.md
@@ -0,0 +1,9 @@
+Bugfix: Fix unshare for EOS storage driver
+
+In the EOS storage driver, the remove acl operation was a no-op.
+After removing a share, the recipient of the share was still able
+to operate on the shared resource.
+Now this has been fixed, removing correctly the ACL from the shared
+resource.
+
+https://github.com/cs3org/reva/pull/3794
diff --git a/pkg/eosclient/eosbinary/eosbinary.go b/pkg/eosclient/eosbinary/eosbinary.go
@@ -314,6 +314,7 @@ func (c *Client) RemoveACL(ctx context.Context, auth, rootAuth eosclient.Authori
 		return err
 	}
 
+	a.Permissions = ""
 	sysACL := a.CitrineSerialize()
 	args := []string{"acl", "--sys"}
 	if finfo.IsDir {