Skip to content

Commit

Permalink
use shares api for space members (#2600)
Browse files Browse the repository at this point in the history
  • Loading branch information
micbar committed Mar 4, 2022
1 parent b8be80d commit b43635e
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 b43635e

Please sign in to comment.