fix(graph): list space members when the public shares cannot be listed

go.mod

@@ -68,7 +68,7 @@ require (
 	github.com/open-policy-agent/opa v1.21.1
 	github.com/opencloud-eu/icap-client v0.0.0-20250930132611-28a2afe62d89
 	github.com/opencloud-eu/libre-graph-api-go v1.0.9-0.20261016073229-17ed3d5dd346
-	github.com/opencloud-eu/reva/v2 v2.52.2-0.20261111125001-fa3c9e289ad6
+	github.com/opencloud-eu/reva/v2 v2.52.2-0.20261113073431-67131be3db65
 	github.com/opensearch-project/opensearch-go/v4 v4.7.3
 	github.com/orcaman/concurrent-map v1.0.0
 	github.com/pkg/errors v0.9.1

go.sum

@@ -761,8 +761,8 @@ github.com/opencloud-eu/icap-client v0.0.0-20250930132611-28a2afe62d89 h1:W1ms+l
 github.com/opencloud-eu/icap-client v0.0.0-20250930132611-28a2afe62d89/go.mod h1:vigJkNss1N2QEceCuNw/ullDehncuJNFB6mEnzfq9UI=
 github.com/opencloud-eu/libre-graph-api-go v1.0.9-0.20261016073229-17ed3d5dd346 h1:3ptbiFhSK4fzY/rC3idcYUveiwepMoOov1sBpc7Rg3k=
 github.com/opencloud-eu/libre-graph-api-go v1.0.9-0.20261016073229-17ed3d5dd346/go.mod h1:lTM8JeGblNpoMySTW7Lui2+c5TTLI95mwxtdUIHHrhU=
-github.com/opencloud-eu/reva/v2 v2.52.2-0.20261111125001-fa3c9e289ad6 h1:EvMm2GuJwm2f7C6CgIulV4ijksRhXj4bgwI5BSfHTxo=
-github.com/opencloud-eu/reva/v2 v2.52.2-0.20261111125001-fa3c9e289ad6/go.mod h1:nKOQFFHQZiBFiiXBqEIc99giHQ/Pe0XYf5cztJhZDYw=
+github.com/opencloud-eu/reva/v2 v2.52.2-0.20261113073431-67131be3db65 h1:u9GVH5aM8Kv8PjKsfxTND8YtRdjuGjviIcNor1L7iV4=
+github.com/opencloud-eu/reva/v2 v2.52.2-0.20261113073431-67131be3db65/go.mod h1:nKOQFFHQZiBFiiXBqEIc99giHQ/Pe0XYf5cztJhZDYw=
 github.com/opencloud-eu/secure v0.0.0-20260312082735-b6f5cb2244e4 h1:l2oB/RctH+t8r7QBj5p8thfEHCM/jF35aAY3WQ3hADI=
 github.com/opencloud-eu/secure v0.0.0-20260312082735-b6f5cb2244e4/go.mod h1:BmF5hyM6tXczk3MpQkFf1hpKSRqCyhqcbiQtiAF7+40=
 github.com/opencontainers/go-digest v1.0.0 h1:apOUWs51W5PlhuyGyz9FCeeBIOUDA/6nW8Oi/yOhh5U=

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

@@ -23,6 +23,7 @@ import (
 	"github.com/go-chi/render"
 	libregraph "github.com/opencloud-eu/libre-graph-api-go"
 	revactx "github.com/opencloud-eu/reva/v2/pkg/ctx"
+	"github.com/opencloud-eu/reva/v2/pkg/errtypes"
 	"github.com/opencloud-eu/reva/v2/pkg/publicshare"
 	"github.com/opencloud-eu/reva/v2/pkg/rgrpc/todo/pool"
 	"github.com/opencloud-eu/reva/v2/pkg/share"
@@ -500,7 +501,11 @@ func (s DriveItemPermissionsService) ListPermissions(ctx context.Context, itemID
 		driveItems, err = s.listPublicShares(ctx, []*link.ListPublicSharesRequest_Filter{
 			publicshare.ResourceIDFilter(itemID),
 		}, driveItems)
-		if err != nil {
+		if err != nil && strings.Contains(err.Error(), errtypes.ERR_ALREADY_EXISTS) {
+			s.logger.Info().Err(err).Str("storageID", itemID.GetStorageId()).Msg("listPublicShares error: " + err.Error())
+		}
+		if err != nil && !strings.Contains(err.Error(), errtypes.ERR_ALREADY_EXISTS) {
+			s.logger.Error().Err(err).Str("storageID", itemID.GetStorageId()).Msg("listPublicShares failed")
 			return collectionOfPermissions, err
 		}
 	}

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

@@ -29,6 +29,7 @@ import (
 
 	roleconversions "github.com/opencloud-eu/reva/v2/pkg/conversions"
 	revactx "github.com/opencloud-eu/reva/v2/pkg/ctx"
+	"github.com/opencloud-eu/reva/v2/pkg/errtypes"
 	"github.com/opencloud-eu/reva/v2/pkg/rgrpc/status"
 	"github.com/opencloud-eu/reva/v2/pkg/rgrpc/todo/pool"
 	"github.com/opencloud-eu/reva/v2/pkg/storagespace"
@@ -512,6 +513,46 @@ var _ = Describe("DriveItemPermissionsService", func() {
 			Expect(len(permissions.LibreGraphPermissionsRolesAllowedValues)).ToNot(BeZero())
 			Expect(len(permissions.Value)).To(Equal(2))
 		})
+		Context("when listing the public shares fails", func() {
+			BeforeEach(func() {
+				statResponse.Info.PermissionSet = roleconversions.NewEditorRole().CS3ResourcePermissions()
+				listSharesResponse.Shares = []*collaboration.Share{
+					{
+						Id: &collaboration.ShareId{OpaqueId: "1"},
+						Permissions: &collaboration.SharePermissions{
+							Permissions: roleconversions.NewViewerRole().CS3ResourcePermissions(),
+						},
+						ResourceId: itemID,
+						Grantee: &provider.Grantee{
+							Type: provider.GranteeType_GRANTEE_TYPE_USER,
+							Id: &provider.Grantee_UserId{
+								UserId: &userpb.UserId{
+									OpaqueId: "user-id",
+								},
+							},
+						},
+					},
+				}
+				gatewayClient.On("Stat", mock.Anything, mock.Anything).Return(statResponse, nil)
+				gatewayClient.On("ListShares", mock.Anything, mock.Anything).Return(listSharesResponse, nil)
+				gatewayClient.On("GetUser", mock.Anything, mock.Anything).Return(getUserResponse, nil).Maybe()
+			})
+			It("still returns the other permissions on an already exists error", func() {
+				gatewayClient.On("ListPublicShares", mock.Anything, mock.Anything).Return(&link.ListPublicSharesResponse{
+					Status: status.NewInternal(ctx, errtypes.AlreadyExists("metadata space").Error()),
+				}, nil)
+				permissions, err := driveItemPermissionsService.ListPermissions(context.Background(), itemID, svc.ListPermissionsQueryOptions{})
+				Expect(err).ToNot(HaveOccurred())
+				Expect(len(permissions.Value)).To(Equal(1))
+			})
+			It("returns the error on any other error", func() {
+				gatewayClient.On("ListPublicShares", mock.Anything, mock.Anything).Return(&link.ListPublicSharesResponse{
+					Status: status.NewInternal(ctx, "transport error"),
+				}, nil)
+				_, err := driveItemPermissionsService.ListPermissions(context.Background(), itemID, svc.ListPermissionsQueryOptions{})
+				Expect(err).To(HaveOccurred())
+			})
+		})
 		It("returns role denied", func() {
 			statResponse.Info.PermissionSet = roleconversions.NewManagerRole().CS3ResourcePermissions()
 			statResponse.Info.Type = provider.ResourceType_RESOURCE_TYPE_CONTAINER

vendor/modules.txt

@@ -1360,7 +1360,7 @@ github.com/opencloud-eu/icap-client
 # github.com/opencloud-eu/libre-graph-api-go v1.0.9-0.20261016073229-17ed3d5dd346
 ## explicit; go 1.23
 github.com/opencloud-eu/libre-graph-api-go
-# github.com/opencloud-eu/reva/v2 v2.52.2-0.20261111125001-fa3c9e289ad6
+# github.com/opencloud-eu/reva/v2 v2.52.2-0.20261113073431-67131be3db65
 ## explicit; go 1.26.0
 github.com/opencloud-eu/reva/v2/cmd/revad/internal/grace
 github.com/opencloud-eu/reva/v2/cmd/revad/runtime