Skip to content

Commit

Permalink
use shares api for space members
Browse files Browse the repository at this point in the history
  • Loading branch information
micbar committed Mar 4, 2022
1 parent 87bfb8d commit 3bc4898
Show file tree
Hide file tree
Showing 4 changed files with 278 additions and 144 deletions.
5 changes: 5 additions & 0 deletions changelog/unreleased/use-share-api-spaces.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Change: Use the cs3 share api to manage spaces

We now use the cs3 share Api to manage the space roles. We do not send the request to the share manager, the permissions are stored in the storage provider

https://github.com/cs3org/reva/pull/2600
2 changes: 1 addition & 1 deletion internal/grpc/services/gateway/ocmshareprovider.go
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ func (s *svc) CreateOCMShare(ctx context.Context, req *ocm.CreateOCMShareRequest

// TODO(labkode): if both commits are enabled they could be done concurrently.
if s.c.CommitShareToStorageGrant {
addGrantStatus, err := s.addGrant(ctx, req.ResourceId, req.Grant.Grantee, req.Grant.Permissions.Permissions)
addGrantStatus, err := s.addGrant(ctx, req.ResourceId, req.Grant.Grantee, req.Grant.Permissions.Permissions, nil)
if err != nil {
return nil, errors.Wrap(err, "gateway: error adding OCM grant to storage")
}
Expand Down
Loading

0 comments on commit 3bc4898

Please sign in to comment.