tree: Introduce APE checks in `verifyClient` #1090

Merged

fyrchik merged 3 commits from aarifullin/frostfs-node:fix/tree_svc_ape into master

2024-04-15 07:45:50 +00:00

aarifullin commented

2024-04-11 15:15:41 +00:00

Collaborator

Update go.mod
Move common ape request and resource implementations to pkg/ape/request package. Also, slightly refactor object and container service.
Introduce workaround for APE checks in tree service.

- Update `go.mod` - Move common ape request and resource implementations to `pkg/ape/request` package. Also, slightly refactor object and container service. - Introduce workaround for APE checks in tree service.

aarifullin added 3 commits 2024-04-11 15:15:46 +00:00

7805069f9d [#XX] go.mod: Update policy-engine version

Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>

f26f3ded28 [#XX] ape: Move ape request and resource implementations to common package

Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>

Vulncheck / Vulncheck (pull_request) Successful in 1m13s Details

Tests and linters / Staticcheck (pull_request) Failing after 17m18s Details

DCO action / DCO (pull_request) Failing after 23m7s Details

Tests and linters / gopls check (pull_request) Failing after 25m21s Details

Tests and linters / Tests with -race (pull_request) Failing after 26m2s Details

Build / Build Components (1.20) (pull_request) Successful in 31m30s Details

Build / Build Components (1.21) (pull_request) Successful in 31m30s Details

Tests and linters / Lint (pull_request) Successful in 33m17s Details

Tests and linters / Tests (1.21) (pull_request) Failing after 33m29s Details

Tests and linters / Tests (1.20) (pull_request) Failing after 33m32s Details

7a26865d6e [#XX] tree: Make workaround for APE checks

* Make `verifyClient` method perform APE check if a container
  was created with zero-filled basic ACL.
* Object verbs are used in APE, until tree verbs are introduced.

Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>

aarifullin changed title from ~~tree: Introduce APE checks in `verifyClient`~~ to WIP: tree: Introduce APE checks in `verifyClient`

2024-04-11 15:20:10 +00:00

aarifullin force-pushed fix/tree_svc_ape from 7a26865d6e to cadf0a40c7

2024-04-11 15:33:36 +00:00

Compare

aarifullin changed title from ~~WIP: tree: Introduce APE checks in `verifyClient`~~ to tree: Introduce APE checks in `verifyClient`

2024-04-11 15:33:55 +00:00

aarifullin force-pushed fix/tree_svc_ape from cadf0a40c7 to b75e2a2709

2024-04-11 15:36:23 +00:00

Compare

aarifullin requested review from storage-core-committers 2024-04-11 16:09:52 +00:00

aarifullin requested review from storage-core-developers 2024-04-11 16:11:50 +00:00