Fix UpdateUserShare when just the expiration date is update

When updating the Grant we need to use the Share as returned in the UpdateUserShareResponse. The Share from the incoming request might just contain a subset of the attributes.
cs3org · Dec 5, 2023 · 83fe141 · 83fe141
1 parent b9486a8
commit 83fe141
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/changelog/unreleased/fix-update-userhare.md b/changelog/unreleased/fix-update-userhare.md
@@ -0,0 +1,6 @@
+Bugfix: Allow UpdateUserShare() to update just the expiration date
+
+The UpdateUserShare Request now works if it just contains an update of the
+expiration date.
+
+https://github.com/cs3org/reva/pull/xxxx
diff --git a/internal/grpc/services/gateway/usershareprovider.go b/internal/grpc/services/gateway/usershareprovider.go
@@ -123,9 +123,9 @@ func (s *svc) updateShare(ctx context.Context, req *collaboration.UpdateShareReq
 	if s.c.CommitShareToStorageGrant {
 		creator := ctxpkg.ContextMustGetUser(ctx)
 		grant := &provider.Grant{
-			Grantee:     req.GetShare().GetGrantee(),
-			Permissions: req.GetShare().GetPermissions().GetPermissions(),
-			Expiration:  req.GetShare().GetExpiration(),
+			Grantee:     res.GetShare().GetGrantee(),
+			Permissions: res.GetShare().GetPermissions().GetPermissions(),
+			Expiration:  res.GetShare().GetExpiration(),
 			Creator:     creator.GetId(),
 		}
 		updateGrantStatus, err := s.updateGrant(ctx, res.GetShare().GetResourceId(), grant, nil)