fix(sharing-ng): align sharedWithMe driveItem IDs

The IDs of the driveItems returned by the 'sharedWithMe' endpoint
should match the ids returned by the webdav PROPFIND response on
the share jail.

Fixes: #8420

changelog/unreleased/sharing-ng-driveitemid.md

@@ -0,0 +1,9 @@
+Bugfix: graph/sharedWithMe align IDs with webdav response
+
+The IDs of the driveItems returned by the 'graph/v1beta1/me/drive/sharedWithMe'
+endpoint are now aligned with the IDs returned in the PROPFIND response
+of the webdav service.
+
+https://github.com/owncloud/ocis/pull/8467
+https://github.com/owncloud/ocis/issues/8420
+https://github.com/owncloud/ocis/issues/8080

services/graph/pkg/service/v0/sharedwithme.go

@@ -92,25 +92,24 @@ func (g Graph) cs3ReceivedSharesToDriveItems(ctx context.Context, receivedShares
 
 		group.Go(func() error {
 			var err error // redeclare
-			resourceID := receivedShares[0].GetShare().GetResourceId()
 			shareStat, err := doStat(receivedShares[0].GetShare().GetResourceId())
 			if shareStat == nil || err != nil {
 				return err
 			}
 
 			driveItem := libregraph.NewDriveItem()
 
-			// The id of the driveItem will be the composed of the StorageID and the SpaceID of the sharestorage
-			// appended with the ResourceID of the shared resource
-			// '<sharestorageid>$<sharespaceid>!<resource's storageid>:<resource's spaceid>:<resource's opaque id>'
-			driveItem.SetId(storagespace.FormatResourceID(storageprovider.ResourceId{
-				StorageId: utils.ShareStorageProviderID,
-				OpaqueId:  resourceID.GetStorageId() + ":" + resourceID.GetSpaceId() + ":" + resourceID.GetOpaqueId(),
-				SpaceId:   utils.ShareStorageSpaceID,
-			}))
 			permissions := make([]libregraph.Permission, 0, len(receivedShares))
 
+			var oldestReceivedShare *collaboration.ReceivedShare
 			for _, receivedShare := range receivedShares {
+				switch {
+				case oldestReceivedShare == nil:
+					fallthrough
+				case utils.TSToTime(receivedShare.GetShare().GetCtime()).Before(utils.TSToTime(oldestReceivedShare.GetShare().GetCtime())):
+					oldestReceivedShare = receivedShare
+				}
+
 				permission, err := g.cs3ReceivedShareToLibreGraphPermissions(ctx, receivedShare)
 				if err != nil {
 					return err
@@ -150,6 +149,23 @@ func (g Graph) cs3ReceivedSharesToDriveItems(ctx context.Context, receivedShares
 
 			}
 
+			// To stay compatible with the usershareprovider and the webdav
+			// service the id of the driveItem is composed of the StorageID and
+			// SpaceID of the sharestorage appended with the opaque ID of
+			// the oldest share for the resource:
+			// '<sharestorageid>$<sharespaceid>!<share-opaque-id>
+			// Note: This means that the driveitem ID will change when the oldest
+			//   shared is removed. It would be good to have are more stable ID here (e.g.
+			//   derived from the shared resource's ID. But as we need to use the same
+			//   ID across all services this means we needed to make similar adjustments
+			//   to the sharejail (usershareprovider, webdav). Which we can't currently do
+			//   as some clients rely on the IDs used there having a special format.
+			driveItem.SetId(storagespace.FormatResourceID(storageprovider.ResourceId{
+				StorageId: utils.ShareStorageProviderID,
+				OpaqueId:  oldestReceivedShare.GetShare().GetId().GetOpaqueId(),
+				SpaceId:   utils.ShareStorageSpaceID,
+			}))
+
 			if !driveItem.HasUIHidden() {
 				driveItem.SetUIHidden(false)
 			}

services/graph/pkg/service/v0/sharedwithme_test.go

@@ -158,6 +158,7 @@ var _ = Describe("SharedWithMe", func() {
 								Permissions: roleconversions.NewViewerRole(true).CS3ResourcePermissions(),
 							},
 							Creator: getUserResponseShareCreator.User.Id,
+							Ctime:   utils.TSNow(),
 						},
 						MountPoint: &providerv1beta1.Reference{
 							ResourceId: &providerv1beta1.ResourceId{
@@ -213,6 +214,7 @@ var _ = Describe("SharedWithMe", func() {
 				Hidden: true,
 				Share: &collaborationv1beta1.Share{
 					ResourceId: toResourceID("7$8!9"),
+					Ctime:      utils.TSNow(),
 				},
 			})
 
@@ -405,6 +407,7 @@ var _ = Describe("SharedWithMe", func() {
 						Permissions: roleconversions.NewViewerRole(true).CS3ResourcePermissions(),
 					},
 					Creator: getUserResponseShareCreator.User.Id,
+					Ctime:   utils.TSNow(),
 					Grantee: &providerv1beta1.Grantee{
 						Type: providerv1beta1.GranteeType_GRANTEE_TYPE_GROUP,
 						Id: &providerv1beta1.Grantee_GroupId{