From 894860fcbbc36c4f1dea660871c0067545a31bc6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=B6rn=20Friedrich=20Dreyer?= Date: Wed, 23 Feb 2022 15:57:15 +0000 Subject: [PATCH 1/7] update reva to include decomposedfs nodes-per-space MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Jörn Friedrich Dreyer --- go.mod | 2 + go.sum | 4 +- graph/pkg/service/v0/drives.go | 3 ++ ocis-pkg/metadata_storage/metadata_storage.go | 3 ++ settings/pkg/service/v0/settings.go | 45 +++++++++++++++++++ .../pkg/command/storagedrivers/metadata.go | 5 +-- storage/pkg/command/storagedrivers/user.go | 6 +-- storage/pkg/config/config.go | 29 +++--------- storage/pkg/config/defaultconfig.go | 24 +++++----- 9 files changed, 74 insertions(+), 47 deletions(-) diff --git a/go.mod b/go.mod index ceb98782107..9c58124da5a 100644 --- a/go.mod +++ b/go.mod @@ -275,3 +275,5 @@ require ( // we need to use a fork to make the windows build pass replace github.com/pkg/xattr => github.com/micbar/xattr v0.4.6-0.20220215112335-88e74d648fb7 + +replace github.com/cs3org/reva => github.com/butonic/reva v0.0.0-20220225081521-7e11e353d614 diff --git a/go.sum b/go.sum index b82561e52cd..da4a7e81374 100644 --- a/go.sum +++ b/go.sum @@ -261,6 +261,8 @@ github.com/bmizerany/pat v0.0.0-20210406213842-e4b6760bdd6f/go.mod h1:8rLXio+Wji github.com/boombuler/barcode v1.0.1-0.20190219062509-6c824513bacc/go.mod h1:paBWMcWSl3LHKBqUq+rly7CNSldXjb2rDl3JlRe0mD8= github.com/boombuler/barcode v1.0.1 h1:NDBbPmhS+EqABEs5Kg3n/5ZNjy73Pz7SIV+KCeqyXcs= github.com/boombuler/barcode v1.0.1/go.mod h1:paBWMcWSl3LHKBqUq+rly7CNSldXjb2rDl3JlRe0mD8= +github.com/butonic/reva v0.0.0-20220225081521-7e11e353d614 h1:sePWKHM82t2+luqxcquLWXgNoud6SlJf12vGju43vhI= +github.com/butonic/reva v0.0.0-20220225081521-7e11e353d614/go.mod h1:M+Nausi93NaMDoinwiUX/Ekgrp/RrYv715FG5KrIarY= github.com/c-bata/go-prompt v0.2.5/go.mod h1:vFnjEGDIIA/Lib7giyE4E9c50Lvl8j0S+7FVlAwDAVw= github.com/cenkalti/backoff v2.2.1+incompatible h1:tNowT99t7UNflLxfYYSlKYsBpXdEet03Pg2g16Swow4= github.com/cenkalti/backoff v2.2.1+incompatible/go.mod h1:90ReRw6GdpyfrHakVjL/QHaoyV4aDUVVkXQJJJ3NXXM= @@ -342,8 +344,6 @@ github.com/crewjam/saml v0.4.5/go.mod h1:qCJQpUtZte9R1ZjUBcW8qtCNlinbO363ooNl02S github.com/cs3org/cato v0.0.0-20200828125504-e418fc54dd5e/go.mod h1:XJEZ3/EQuI3BXTp/6DUzFr850vlxq11I6satRtz0YQ4= github.com/cs3org/go-cs3apis v0.0.0-20220126114148-64c025ccdd19 h1:1jqPH58jCxvbaJ9WLIJ7W2/m622bWS6ChptzljSG6IQ= github.com/cs3org/go-cs3apis v0.0.0-20220126114148-64c025ccdd19/go.mod h1:UXha4TguuB52H14EMoSsCqDj7k8a/t7g4gVP+bgY5LY= -github.com/cs3org/reva v1.16.1-0.20220224115936-e17c65a48a5a h1:LgwfJ4N+ekLRFOK2vwVKbgtnQYYcrNiMtu5kekJvJ54= -github.com/cs3org/reva v1.16.1-0.20220224115936-e17c65a48a5a/go.mod h1:M+Nausi93NaMDoinwiUX/Ekgrp/RrYv715FG5KrIarY= github.com/cubewise-code/go-mime v0.0.0-20200519001935-8c5762b177d8 h1:Z9lwXumT5ACSmJ7WGnFl+OMLLjpz5uR2fyz7dC255FI= github.com/cubewise-code/go-mime v0.0.0-20200519001935-8c5762b177d8/go.mod h1:4abs/jPXcmJzYoYGF91JF9Uq9s/KL5n1jvFDix8KcqY= github.com/cyberdelia/templates v0.0.0-20141128023046-ca7fffd4298c/go.mod h1:GyV+0YP4qX0UQ7r2MoYZ+AvYDp12OF5yg4q8rGnyNh4= diff --git a/graph/pkg/service/v0/drives.go b/graph/pkg/service/v0/drives.go index b5b185be256..ec1ae4525fc 100644 --- a/graph/pkg/service/v0/drives.go +++ b/graph/pkg/service/v0/drives.go @@ -418,6 +418,9 @@ func (g Graph) ListStorageSpacesWithFilters(ctx context.Context, filters []*stor } func (g Graph) cs3StorageSpaceToDrive(baseURL *url.URL, space *storageprovider.StorageSpace) (*libregraph.Drive, error) { + if space.Root == nil { + return nil, fmt.Errorf("space has no root") + } rootID := resourceid.OwnCloudResourceIDWrap(space.Root) var permissions []libregraph.Permission diff --git a/ocis-pkg/metadata_storage/metadata_storage.go b/ocis-pkg/metadata_storage/metadata_storage.go index 51c293ec67a..b99a813ac59 100644 --- a/ocis-pkg/metadata_storage/metadata_storage.go +++ b/ocis-pkg/metadata_storage/metadata_storage.go @@ -89,6 +89,9 @@ func (ms MetadataStorage) SimpleUpload(ctx context.Context, uploadpath string, c if err != nil { return err } + if res.Status.Code != v1beta11.Code_CODE_OK { + return errtypes.NewErrtypeFromStatus(res.Status) + } var endpoint string diff --git a/settings/pkg/service/v0/settings.go b/settings/pkg/service/v0/settings.go index 454becfbac7..338e4d44daa 100644 --- a/settings/pkg/service/v0/settings.go +++ b/settings/pkg/service/v0/settings.go @@ -6,6 +6,9 @@ import ( ) const ( + // BundleUUIDRoleMetadata represents the metadata user role + BundleUUIDRoleMetadata = "95cb8724-03b2-11eb-a0a6-c33ef8ef53ad" + // BundleUUIDRoleAdmin represents the admin role BundleUUIDRoleAdmin = "71881883-1768-46bd-a24d-a356a2afdf7f" @@ -366,6 +369,24 @@ func generatePermissionRequests() []*settingssvc.AddSettingToBundleRequest { }, }, }, + { + BundleId: BundleUUIDRoleUser, + Setting: &settingsmsg.Setting{ + Id: CreateSpacePermissionID, + Name: CreateSpacePermissionName, + DisplayName: "Create own Space", + Description: "This permission allows to create a space owned by the current user.", + Resource: &settingsmsg.Resource{ + Type: settingsmsg.Resource_TYPE_SYSTEM, // TODO resource type space? self? me? own? + }, + Value: &settingsmsg.Setting_PermissionValue{ + PermissionValue: &settingsmsg.Permission{ + Operation: settingsmsg.Permission_OPERATION_CREATE, + Constraint: settingsmsg.Permission_CONSTRAINT_OWN, + }, + }, + }, + }, { BundleId: BundleUUIDRoleAdmin, Setting: &settingsmsg.Setting{ @@ -402,11 +423,35 @@ func generatePermissionRequests() []*settingssvc.AddSettingToBundleRequest { }, }, }, + { + BundleId: BundleUUIDRoleMetadata, + Setting: &settingsmsg.Setting{ + Id: CreateSpacePermissionID, + Name: CreateSpacePermissionName, + DisplayName: "Create own Space", + Description: "This permission allows to create a space owned by the current user.", + Resource: &settingsmsg.Resource{ + Type: settingsmsg.Resource_TYPE_SYSTEM, // TODO resource type space? self? me? own? + }, + Value: &settingsmsg.Setting_PermissionValue{ + PermissionValue: &settingsmsg.Permission{ + Operation: settingsmsg.Permission_OPERATION_CREATE, + Constraint: settingsmsg.Permission_CONSTRAINT_OWN, + }, + }, + }, + }, } } func defaultRoleAssignments() []*settingsmsg.UserRoleAssignment { return []*settingsmsg.UserRoleAssignment{ + // accounts service user for the metadata user is allowed to create spaces + + { + AccountUuid: "95cb8724-03b2-11eb-a0a6-c33ef8ef53ad", + RoleId: BundleUUIDRoleAdmin, + }, // default admin users { AccountUuid: "058bff95-6708-4fe5-91e4-9ea3d377588b", diff --git a/storage/pkg/command/storagedrivers/metadata.go b/storage/pkg/command/storagedrivers/metadata.go index 6aa840296e1..445d386c2c5 100644 --- a/storage/pkg/command/storagedrivers/metadata.go +++ b/storage/pkg/command/storagedrivers/metadata.go @@ -50,11 +50,10 @@ func MetadataDrivers(cfg *config.Config) map[string]interface{} { }, "ocis": map[string]interface{}{ "root": cfg.Reva.MetadataStorage.OCIS.Root, - "enable_home": false, "user_layout": cfg.Reva.MetadataStorage.OCIS.UserLayout, "treetime_accounting": false, "treesize_accounting": false, - "owner": cfg.Reva.MetadataStorage.OCIS.ServiceUserUUID, // the accounts service system account uuid + "permissionssvc": cfg.Reva.Permissions.Endpoint, }, "s3": map[string]interface{}{ "region": cfg.Reva.MetadataStorage.S3.Region, @@ -69,7 +68,7 @@ func MetadataDrivers(cfg *config.Config) map[string]interface{} { "user_layout": cfg.Reva.MetadataStorage.S3NG.UserLayout, "treetime_accounting": false, "treesize_accounting": false, - "owner": cfg.Reva.MetadataStorage.S3NG.ServiceUserUUID, // the accounts service system account uuid + "permissionssvc": cfg.Reva.Permissions.Endpoint, "s3.region": cfg.Reva.MetadataStorage.S3NG.Region, "s3.access_key": cfg.Reva.MetadataStorage.S3NG.AccessKey, "s3.secret_key": cfg.Reva.MetadataStorage.S3NG.SecretKey, diff --git a/storage/pkg/command/storagedrivers/user.go b/storage/pkg/command/storagedrivers/user.go index 11b60a0312c..412f08deef6 100644 --- a/storage/pkg/command/storagedrivers/user.go +++ b/storage/pkg/command/storagedrivers/user.go @@ -90,12 +90,11 @@ func UserDrivers(cfg *config.Config) map[string]interface{} { }, "ocis": map[string]interface{}{ "root": cfg.Reva.UserStorage.OCIS.Root, - "enable_home": false, "user_layout": cfg.Reva.UserStorage.OCIS.UserLayout, "share_folder": cfg.Reva.UserStorage.OCIS.ShareFolder, "treetime_accounting": true, "treesize_accounting": true, - "owner": cfg.Reva.UserStorage.OCIS.ServiceUserUUID, // the accounts service system account uuid + "permissionssvc": cfg.Reva.Permissions.Endpoint, }, "s3": map[string]interface{}{ "enable_home": false, @@ -108,12 +107,11 @@ func UserDrivers(cfg *config.Config) map[string]interface{} { }, "s3ng": map[string]interface{}{ "root": cfg.Reva.UserStorage.S3NG.Root, - "enable_home": false, "user_layout": cfg.Reva.UserStorage.S3NG.UserLayout, "share_folder": cfg.Reva.UserStorage.S3NG.ShareFolder, "treetime_accounting": true, "treesize_accounting": true, - "owner": cfg.Reva.UserStorage.S3NG.ServiceUserUUID, // the accounts service system account uuid + "permissionssvc": cfg.Reva.Permissions.Endpoint, "s3.region": cfg.Reva.UserStorage.S3NG.Region, "s3.access_key": cfg.Reva.UserStorage.S3NG.AccessKey, "s3.secret_key": cfg.Reva.UserStorage.S3NG.SecretKey, diff --git a/storage/pkg/config/config.go b/storage/pkg/config/config.go index 7541633f58b..086f4b9ef23 100644 --- a/storage/pkg/config/config.go +++ b/storage/pkg/config/config.go @@ -307,8 +307,6 @@ type DriverEOS struct { // DriverOCIS defines the available oCIS storage driver configuration. type DriverOCIS struct { DriverCommon - - ServiceUserUUID string `ocisConfig:"service_user_uuid"` } // DriverOwnCloudSQL defines the available ownCloudSQL storage driver configuration. @@ -338,12 +336,11 @@ type DriverS3 struct { type DriverS3NG struct { DriverCommon - ServiceUserUUID string `ocisConfig:"service_user_uuid"` - Region string `ocisConfig:"region"` - AccessKey string `ocisConfig:"access_key"` - SecretKey string `ocisConfig:"secret_key"` - Endpoint string `ocisConfig:"endpoint"` - Bucket string `ocisConfig:"bucket"` + Region string `ocisConfig:"region"` + AccessKey string `ocisConfig:"access_key"` + SecretKey string `ocisConfig:"secret_key"` + Endpoint string `ocisConfig:"endpoint"` + Bucket string `ocisConfig:"bucket"` } // OIDC defines the available OpenID Connect configuration. @@ -1531,10 +1528,6 @@ func structMappings(cfg *Config) []shared.EnvBinding { EnvVars: []string{"STORAGE_USERS_DRIVER_OCIS_SHARE_FOLDER"}, Destination: &cfg.Reva.UserStorage.OCIS.ShareFolder, }, - { - EnvVars: []string{"STORAGE_USERS_DRIVER_OCIS_SERVICE_USER_UUID"}, - Destination: &cfg.Reva.UserStorage.OCIS.ServiceUserUUID, - }, // driver owncloud sql { EnvVars: []string{"STORAGE_USERS_DRIVER_OWNCLOUDSQL_DATADIR"}, @@ -1604,10 +1597,6 @@ func structMappings(cfg *Config) []shared.EnvBinding { EnvVars: []string{"STORAGE_USERS_DRIVER_S3NG_LAYOUT"}, Destination: &cfg.Reva.UserStorage.S3NG.UserLayout, }, - { - EnvVars: []string{"STORAGE_USERS_DRIVER_S3NG_SERVICE_USER_UUID"}, - Destination: &cfg.Reva.UserStorage.S3NG.ServiceUserUUID, - }, { EnvVars: []string{"STORAGE_USERS_DRIVER_S3NG_SHARE_FOLDER"}, Destination: &cfg.Reva.UserStorage.S3NG.ShareFolder, @@ -1718,10 +1707,6 @@ func structMappings(cfg *Config) []shared.EnvBinding { EnvVars: []string{"STORAGE_METADATA_DRIVER_OCIS_LAYOUT"}, Destination: &cfg.Reva.MetadataStorage.OCIS.UserLayout, }, - { - EnvVars: []string{"STORAGE_METADATA_DRIVER_OCIS_SERVICE_USER_UUID"}, - Destination: &cfg.Reva.MetadataStorage.OCIS.ServiceUserUUID, - }, // metadata driver s3 { @@ -1754,10 +1739,6 @@ func structMappings(cfg *Config) []shared.EnvBinding { EnvVars: []string{"STORAGE_METADATA_DRIVER_S3NG_LAYOUT"}, Destination: &cfg.Reva.MetadataStorage.S3NG.UserLayout, }, - { - EnvVars: []string{"STORAGE_METADATA_DRIVER_S3NG_SERVICE_USER_UUID"}, - Destination: &cfg.Reva.MetadataStorage.S3NG.ServiceUserUUID, - }, { EnvVars: []string{"STORAGE_METADATA_DRIVER_S3NG_REGION"}, Destination: &cfg.Reva.MetadataStorage.S3NG.Region, diff --git a/storage/pkg/config/defaultconfig.go b/storage/pkg/config/defaultconfig.go index e186f492096..9ec857bbc5a 100644 --- a/storage/pkg/config/defaultconfig.go +++ b/storage/pkg/config/defaultconfig.go @@ -141,12 +141,11 @@ func DefaultConfig() *Config { UserLayout: defaultUserLayout, EnableHome: false, }, - ServiceUserUUID: defaultServiceUserUUID, - Region: "default", - AccessKey: "", - SecretKey: "", - Endpoint: "", - Bucket: "", + Region: "default", + AccessKey: "", + SecretKey: "", + Endpoint: "", + Bucket: "", }, OCIS: DriverOCIS{ DriverCommon: DriverCommon{ @@ -154,7 +153,6 @@ func DefaultConfig() *Config { ShareFolder: defaultShareFolder, UserLayout: defaultUserLayout, }, - ServiceUserUUID: defaultServiceUserUUID, }, }, MetadataStorage: StorageConfig{ @@ -197,12 +195,11 @@ func DefaultConfig() *Config { UserLayout: defaultUserLayout, EnableHome: false, }, - ServiceUserUUID: defaultServiceUserUUID, - Region: "default", - AccessKey: "", - SecretKey: "", - Endpoint: "", - Bucket: "", + Region: "default", + AccessKey: "", + SecretKey: "", + Endpoint: "", + Bucket: "", }, OCIS: DriverOCIS{ DriverCommon: DriverCommon{ @@ -211,7 +208,6 @@ func DefaultConfig() *Config { UserLayout: defaultUserLayout, EnableHome: false, }, - ServiceUserUUID: defaultServiceUserUUID, }, }, Frontend: FrontendPort{ From ee89c3d9c3226616ca41cc882a29c044ed4619f7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=B6rn=20Friedrich=20Dreyer?= Date: Fri, 25 Feb 2022 11:23:05 +0000 Subject: [PATCH 2/7] check create space with own constraint MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Jörn Friedrich Dreyer --- graph/pkg/service/v0/drives.go | 25 +++++++++++++++++++------ 1 file changed, 19 insertions(+), 6 deletions(-) diff --git a/graph/pkg/service/v0/drives.go b/graph/pkg/service/v0/drives.go index ec1ae4525fc..fb983f7232f 100644 --- a/graph/pkg/service/v0/drives.go +++ b/graph/pkg/service/v0/drives.go @@ -25,6 +25,7 @@ import ( libregraph "github.com/owncloud/libre-graph-api-go" "github.com/owncloud/ocis/graph/pkg/service/v0/errorcode" "github.com/owncloud/ocis/ocis-pkg/service/grpc" + v0 "github.com/owncloud/ocis/protogen/gen/ocis/messages/settings/v0" settingssvc "github.com/owncloud/ocis/protogen/gen/ocis/services/settings/v0" settingsServiceExt "github.com/owncloud/ocis/settings/pkg/service/v0" merrors "go-micro.dev/v4/errors" @@ -151,6 +152,22 @@ func (g Graph) GetSingleDrive(w http.ResponseWriter, r *http.Request) { } } +func canCreateSpace(ctx context.Context, ownPersonalHome bool) bool { + s := settingssvc.NewPermissionService("com.owncloud.api.settings", grpc.DefaultClient) + + pr, err := s.GetPermissionByID(ctx, &settingssvc.GetPermissionByIDRequest{ + PermissionId: settingsServiceExt.CreateSpacePermissionID, + }) + if err != nil || pr.Permission == nil { + return false + } + // TODO @C0rby shouldn't the permissions service check this? aka shouldn't we call CheckPermission? + if pr.Permission.Constraint == v0.Permission_CONSTRAINT_OWN && !ownPersonalHome { + return false + } + return true +} + // CreateDrive creates a storage drive (space). func (g Graph) CreateDrive(w http.ResponseWriter, r *http.Request) { us, ok := ctxpkg.ContextGetUser(r.Context()) @@ -159,12 +176,8 @@ func (g Graph) CreateDrive(w http.ResponseWriter, r *http.Request) { return } - s := settingssvc.NewPermissionService("com.owncloud.api.settings", grpc.DefaultClient) - - _, err := s.GetPermissionByID(r.Context(), &settingssvc.GetPermissionByIDRequest{ - PermissionId: settingsServiceExt.CreateSpacePermissionID, - }) - if err != nil { + // TODO determine if the user tries to create his own personal space and pass that as a boolean + if !canCreateSpace(r.Context(), false) { // if the permission is not existing for the user in context we can assume we don't have it. Return 401. errorcode.GeneralException.Render(w, r, http.StatusUnauthorized, "insufficient permissions to create a space.") return From 6b168eb8f00826de696f0cde6393ac95a72466a6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=B6rn=20Friedrich=20Dreyer?= Date: Fri, 25 Feb 2022 13:12:33 +0000 Subject: [PATCH 3/7] update reva MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Jörn Friedrich Dreyer --- go.mod | 2 +- go.sum | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/go.mod b/go.mod index 9c58124da5a..74cc177881f 100644 --- a/go.mod +++ b/go.mod @@ -276,4 +276,4 @@ require ( // we need to use a fork to make the windows build pass replace github.com/pkg/xattr => github.com/micbar/xattr v0.4.6-0.20220215112335-88e74d648fb7 -replace github.com/cs3org/reva => github.com/butonic/reva v0.0.0-20220225081521-7e11e353d614 +replace github.com/cs3org/reva => github.com/butonic/reva v0.0.0-20220225135442-17afed969f07 diff --git a/go.sum b/go.sum index da4a7e81374..82b36972fd7 100644 --- a/go.sum +++ b/go.sum @@ -261,8 +261,8 @@ github.com/bmizerany/pat v0.0.0-20210406213842-e4b6760bdd6f/go.mod h1:8rLXio+Wji github.com/boombuler/barcode v1.0.1-0.20190219062509-6c824513bacc/go.mod h1:paBWMcWSl3LHKBqUq+rly7CNSldXjb2rDl3JlRe0mD8= github.com/boombuler/barcode v1.0.1 h1:NDBbPmhS+EqABEs5Kg3n/5ZNjy73Pz7SIV+KCeqyXcs= github.com/boombuler/barcode v1.0.1/go.mod h1:paBWMcWSl3LHKBqUq+rly7CNSldXjb2rDl3JlRe0mD8= -github.com/butonic/reva v0.0.0-20220225081521-7e11e353d614 h1:sePWKHM82t2+luqxcquLWXgNoud6SlJf12vGju43vhI= -github.com/butonic/reva v0.0.0-20220225081521-7e11e353d614/go.mod h1:M+Nausi93NaMDoinwiUX/Ekgrp/RrYv715FG5KrIarY= +github.com/butonic/reva v0.0.0-20220225135442-17afed969f07 h1:6vUqaq0cDldnXxQXR8b9NnaTeHzRTRqp1ne0pVX1UQA= +github.com/butonic/reva v0.0.0-20220225135442-17afed969f07/go.mod h1:fdlrnZ0f+UtAdpZfLG+4LM0ZrhT5V8tPEQt6ycYm82c= github.com/c-bata/go-prompt v0.2.5/go.mod h1:vFnjEGDIIA/Lib7giyE4E9c50Lvl8j0S+7FVlAwDAVw= github.com/cenkalti/backoff v2.2.1+incompatible h1:tNowT99t7UNflLxfYYSlKYsBpXdEet03Pg2g16Swow4= github.com/cenkalti/backoff v2.2.1+incompatible/go.mod h1:90ReRw6GdpyfrHakVjL/QHaoyV4aDUVVkXQJJJ3NXXM= From d152121ee97a480c162947a7adc47846a120248c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=B6rn=20Friedrich=20Dreyer?= Date: Fri, 25 Feb 2022 14:54:17 +0000 Subject: [PATCH 4/7] unexpected passed MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Jörn Friedrich Dreyer --- tests/acceptance/expected-failures-API-on-OCIS-storage.md | 2 -- 1 file changed, 2 deletions(-) diff --git a/tests/acceptance/expected-failures-API-on-OCIS-storage.md b/tests/acceptance/expected-failures-API-on-OCIS-storage.md index 180a071e7f6..9c741cfb023 100644 --- a/tests/acceptance/expected-failures-API-on-OCIS-storage.md +++ b/tests/acceptance/expected-failures-API-on-OCIS-storage.md @@ -513,8 +513,6 @@ cannot share a folder with create permission - [apiSharePublicLink1/createPublicLinkShare.feature:375](https://github.com/owncloud/core/blob/master/tests/acceptance/features/apiSharePublicLink1/createPublicLinkShare.feature#L375) - [apiSharePublicLink1/createPublicLinkShare.feature:376](https://github.com/owncloud/core/blob/master/tests/acceptance/features/apiSharePublicLink1/createPublicLinkShare.feature#L376) -- [apiSharePublicLink1/createPublicLinkShare.feature:477](https://github.com/owncloud/core/blob/master/tests/acceptance/features/apiSharePublicLink1/createPublicLinkShare.feature#L477) -- [apiSharePublicLink1/createPublicLinkShare.feature:478](https://github.com/owncloud/core/blob/master/tests/acceptance/features/apiSharePublicLink1/createPublicLinkShare.feature#L478) - [apiSharePublicLink1/createPublicLinkShare.feature:566](https://github.com/owncloud/core/blob/master/tests/acceptance/features/apiSharePublicLink1/createPublicLinkShare.feature#L566) - [apiSharePublicLink1/createPublicLinkShare.feature:567](https://github.com/owncloud/core/blob/master/tests/acceptance/features/apiSharePublicLink1/createPublicLinkShare.feature#L567) - [apiShareManagementBasicToShares/deleteShareFromShares.feature:212](https://github.com/owncloud/core/blob/master/tests/acceptance/features/apiShareManagementBasicToShares/deleteShareFromShares.feature#L212) From c8cf749deb46150d54c1d0a2a603f6b4ee334832 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=B6rn=20Friedrich=20Dreyer?= Date: Tue, 1 Mar 2022 08:53:08 +0000 Subject: [PATCH 5/7] update reva MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Jörn Friedrich Dreyer --- go.mod | 4 +--- go.sum | 4 ++-- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/go.mod b/go.mod index 74cc177881f..70ce6cb0e9e 100644 --- a/go.mod +++ b/go.mod @@ -22,7 +22,7 @@ require ( github.com/blevesearch/bleve/v2 v2.3.1 github.com/coreos/go-oidc/v3 v3.1.0 github.com/cs3org/go-cs3apis v0.0.0-20220126114148-64c025ccdd19 - github.com/cs3org/reva v1.16.1-0.20220224115936-e17c65a48a5a + github.com/cs3org/reva v1.16.1-0.20220301071903-1fd81b097801 github.com/disintegration/imaging v1.6.2 github.com/glauth/glauth/v2 v2.0.0-20211021011345-ef3151c28733 github.com/go-chi/chi/v5 v5.0.7 @@ -275,5 +275,3 @@ require ( // we need to use a fork to make the windows build pass replace github.com/pkg/xattr => github.com/micbar/xattr v0.4.6-0.20220215112335-88e74d648fb7 - -replace github.com/cs3org/reva => github.com/butonic/reva v0.0.0-20220225135442-17afed969f07 diff --git a/go.sum b/go.sum index 82b36972fd7..2c7d70e9b0e 100644 --- a/go.sum +++ b/go.sum @@ -261,8 +261,6 @@ github.com/bmizerany/pat v0.0.0-20210406213842-e4b6760bdd6f/go.mod h1:8rLXio+Wji github.com/boombuler/barcode v1.0.1-0.20190219062509-6c824513bacc/go.mod h1:paBWMcWSl3LHKBqUq+rly7CNSldXjb2rDl3JlRe0mD8= github.com/boombuler/barcode v1.0.1 h1:NDBbPmhS+EqABEs5Kg3n/5ZNjy73Pz7SIV+KCeqyXcs= github.com/boombuler/barcode v1.0.1/go.mod h1:paBWMcWSl3LHKBqUq+rly7CNSldXjb2rDl3JlRe0mD8= -github.com/butonic/reva v0.0.0-20220225135442-17afed969f07 h1:6vUqaq0cDldnXxQXR8b9NnaTeHzRTRqp1ne0pVX1UQA= -github.com/butonic/reva v0.0.0-20220225135442-17afed969f07/go.mod h1:fdlrnZ0f+UtAdpZfLG+4LM0ZrhT5V8tPEQt6ycYm82c= github.com/c-bata/go-prompt v0.2.5/go.mod h1:vFnjEGDIIA/Lib7giyE4E9c50Lvl8j0S+7FVlAwDAVw= github.com/cenkalti/backoff v2.2.1+incompatible h1:tNowT99t7UNflLxfYYSlKYsBpXdEet03Pg2g16Swow4= github.com/cenkalti/backoff v2.2.1+incompatible/go.mod h1:90ReRw6GdpyfrHakVjL/QHaoyV4aDUVVkXQJJJ3NXXM= @@ -344,6 +342,8 @@ github.com/crewjam/saml v0.4.5/go.mod h1:qCJQpUtZte9R1ZjUBcW8qtCNlinbO363ooNl02S github.com/cs3org/cato v0.0.0-20200828125504-e418fc54dd5e/go.mod h1:XJEZ3/EQuI3BXTp/6DUzFr850vlxq11I6satRtz0YQ4= github.com/cs3org/go-cs3apis v0.0.0-20220126114148-64c025ccdd19 h1:1jqPH58jCxvbaJ9WLIJ7W2/m622bWS6ChptzljSG6IQ= github.com/cs3org/go-cs3apis v0.0.0-20220126114148-64c025ccdd19/go.mod h1:UXha4TguuB52H14EMoSsCqDj7k8a/t7g4gVP+bgY5LY= +github.com/cs3org/reva v1.16.1-0.20220301071903-1fd81b097801 h1:FOjP9FbcvD48as7Q7TjOtnaNHlQ5va2IEIed1GWqEag= +github.com/cs3org/reva v1.16.1-0.20220301071903-1fd81b097801/go.mod h1:fdlrnZ0f+UtAdpZfLG+4LM0ZrhT5V8tPEQt6ycYm82c= github.com/cubewise-code/go-mime v0.0.0-20200519001935-8c5762b177d8 h1:Z9lwXumT5ACSmJ7WGnFl+OMLLjpz5uR2fyz7dC255FI= github.com/cubewise-code/go-mime v0.0.0-20200519001935-8c5762b177d8/go.mod h1:4abs/jPXcmJzYoYGF91JF9Uq9s/KL5n1jvFDix8KcqY= github.com/cyberdelia/templates v0.0.0-20141128023046-ca7fffd4298c/go.mod h1:GyV+0YP4qX0UQ7r2MoYZ+AvYDp12OF5yg4q8rGnyNh4= From 3a050d8cdecc116cb80ee239ef1e01a822b41261 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=B6rn=20Friedrich=20Dreyer?= Date: Tue, 1 Mar 2022 09:30:07 +0000 Subject: [PATCH 6/7] remove unused variable MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Jörn Friedrich Dreyer --- storage/pkg/config/defaultconfig.go | 1 - 1 file changed, 1 deletion(-) diff --git a/storage/pkg/config/defaultconfig.go b/storage/pkg/config/defaultconfig.go index 9ec857bbc5a..b66e55f1ac0 100644 --- a/storage/pkg/config/defaultconfig.go +++ b/storage/pkg/config/defaultconfig.go @@ -13,7 +13,6 @@ const ( defaultStorageNamespace = "/users/{{.Id.OpaqueId}}" defaultGatewayAddr = "127.0.0.1:9142" defaultUserLayout = "{{.Id.OpaqueId}}" - defaultServiceUserUUID = "95cb8724-03b2-11eb-a0a6-c33ef8ef53ad" ) func DefaultConfig() *Config { From 6c8682f0bace07b124f370af5659c6f16991eeac Mon Sep 17 00:00:00 2001 From: Florian Schade Date: Tue, 1 Mar 2022 15:05:24 +0100 Subject: [PATCH 7/7] revert reva commit id